Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add logic for differing SLA for order tickets #4274

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

beatrizsavinhas
Copy link
Contributor

@beatrizsavinhas beatrizsavinhas commented Mar 10, 2025

Description

Closes #4153

Added

  • Logic for differing SLA for order tickets

How to prepare for test

  • Deploy branching to stage environment

How to test and Expected test outcome

  • Place express_priority_order with one express case and check that the due dates are in 7 days
  • Place priority_priority_order with one priority case and check that the due dates are in 14 days
  • Place standard_priority_order with one standard and one clinical_trials case and check that the due dates are in 21 days
  • Place research_priority_order with one research case and check that the due dates are in 60 days
  • Place priority_priority_order_with_multiple_cases with one priority case and several other cases with lower priority and check that the due dates are in 14 days
  • Place express_priority_order_with_new_and_existing_samples with one express new case and several other existing cases with lower priority and check that the due dates are in 7 days

Review

  • Tests executed by BSV
  • "Merge and deploy" approved by
    Thanks for filling in who performed the code review and the test!

This version is a

  • MAJOR - when you make incompatible API changes
  • MINOR - when you add functionality in a backwards compatible manner
  • PATCH - when you make backwards compatible bug fixes or documentation/instructions

Implementation Plan

  • Inform to production manager.

@clingen-sthlm clingen-sthlm temporarily deployed to stage March 11, 2025 12:33 Inactive
@clingen-sthlm clingen-sthlm temporarily deployed to stage March 17, 2025 14:25 Inactive
@clingen-sthlm clingen-sthlm temporarily deployed to stage March 18, 2025 09:41 Inactive
@clingen-sthlm clingen-sthlm temporarily deployed to stage March 18, 2025 09:53 Inactive
@clingen-sthlm clingen-sthlm temporarily deployed to stage March 18, 2025 14:14 Inactive
@beatrizsavinhas
Copy link
Contributor Author

beatrizsavinhas commented Mar 18, 2025

Tests

express_priority_order

Screenshot 2025-03-18 at 15 27 40

priority_priority_order

Screenshot 2025-03-18 at 15 28 00

standard_priority_order

Screenshot 2025-03-18 at 15 28 33

research_priority_order

Screenshot 2025-03-18 at 15 34 31

priority_priority_order_with_multiple_cases

Screenshot 2025-03-18 at 15 27 13

express_priority_order_with_new_and_existing_samples

Screenshot 2025-03-18 at 15 37 39

@beatrizsavinhas beatrizsavinhas marked this pull request as ready for review March 18, 2025 14:31
@beatrizsavinhas beatrizsavinhas requested a review from a team as a code owner March 18, 2025 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Set differing SLA for order tickets
2 participants