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

Bug: Delivery of cases where case.name is the same as the sample.name #3745

Open
karlnyr opened this issue Sep 18, 2024 · 6 comments · May be fixed by #3815
Open

Bug: Delivery of cases where case.name is the same as the sample.name #3745

karlnyr opened this issue Sep 18, 2024 · 6 comments · May be fixed by #3815

Comments

@karlnyr
Copy link
Contributor

karlnyr commented Sep 18, 2024

Describe the bug

When one delivers a case today the idea is that we create the following two directories under the ticket folder in the customer's inbox. For example, if we have a case with an case.internal_id funnycat and a case.name case-name, with a sample with the sample.name sample-name:

cg deliver case --dry-run -c funnycat # Delivers the case to the inbox, thus we get
ll /path/to/inbox/<ticket_id>/
case-name
sample-name

Where we today put everything "case" related into the folder named case-name and everything "sample" related into the folder named sample-name.

However, users are able to place the order of a case, where the case.name is the same as sample.name. This makes the delivery structure different. For example, if we change the sample.name to case-name above we would get:

cg deliver case --dry-run -c funnycat # Delivers the case to the inbox
ll /path/to/inbox/<ticket_id>/
case-name # This folder now contains both the case files and the sample files

As the case.name and sample.name are the same we redirect the files to the same location. This is not the end of the world, but confusing to our customers if they expect the sample files to be in the sample folder, but the sample folder is now the case folder.

To Reproduce
Steps to reproduce the behavior:

  1. `cg deliver case --dry-run -c <case_where_case.name=related_sample.name>

Expected behavior

I expect that we have a consistent format of delivery. Either we put everything into one folder, or keep them separate as is. But if we want to keep them separate we also have to remove the possibility of placing orders where the case.name is equal to the sample.name

Additional context
Add any other context about the problem here.

@karlnyr
Copy link
Contributor Author

karlnyr commented Sep 18, 2024

I am uncertain of the effort, to be honest, it feels like it could vary a bit.

@diitaz93
Copy link
Contributor

Acceptance criteria by sysdev

The sample name can not be equal to the case name

@diitaz93
Copy link
Contributor

Awaiting refinement by @Clinical-Genomics/prodbioinfo

@karlnyr
Copy link
Contributor Author

karlnyr commented Sep 19, 2024

We want this 👍

@Vince-janv
Copy link
Contributor

Technical refinement

  • In the improve-order-flow, add a rule for orders with cases preventing a sample being linked to a case with the same name

@diitaz93
Copy link
Contributor

Implemented in #4022. Will close when dev branch is merged

@Vince-janv Vince-janv linked a pull request Dec 13, 2024 that will close this issue
15 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants