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.

Sign up for GitHub

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

Jump to bottom

Import mail: series order of exam is wrong #2491

Closed
michaelkain opened this issue Nov 7, 2024 · 1 comment · Fixed by #2528 or #2562
Closed

Import mail: series order of exam is wrong #2491

michaelkain opened this issue Nov 7, 2024 · 1 comment · Fixed by #2528 or #2562
Assignees
@DuckflipXYZ
Labels
backend bug import minor WorkInProgress

Comments

@michaelkain
Copy link
Contributor

Hi Guéwen,
I hope you are doing great.
This is a minor issue, that would be great to solve and start with:
When I receive an import mail, sent by Shanoir server, after an import,
in the mail the series (list of dataset acquisitions) is not ordered by sorting_index (seriesNumber).
This his confusing for our users, please fix this. Thank you and with kind regards, Michael
DuckflipXYZ added a commit to DuckflipXYZ/shanoir-ng that referenced this issue Dec 2, 2024
@DuckflipXYZ
fli-iam#2491 : Sorting acquisitions in mail per sorting index when im…
345dd83 
…ported from PACS
michaelkain added a commit that referenced this issue Dec 2, 2024
@michaelkain
Merge pull request #2528 from DuckflipXYZ/2491
695f5ba 
#2491 : Sorting acquisitions in mail per sorting index when imported from PACS
@DuckflipXYZ DuckflipXYZ linked a pull request Dec 3, 2024 that will close this issue
#2491 : Sorting acquisitions in mail per sorting index when imported from PACS #2528
Merged
@DuckflipXYZ
Copy link
Collaborator

Merged, but needs to be test in qualif (asap, currently impossible due to connexion issue)

DuckflipXYZ added a commit to DuckflipXYZ/shanoir-ng that referenced this issue Dec 20, 2024
@DuckflipXYZ
fli-iam#2491 : HashMap -> LinkedHashMap for sorting preservation
a65d2a2
@DuckflipXYZ DuckflipXYZ linked a pull request Dec 20, 2024 that will close this issue
#2491 : HashMap -> LinkedHashMap for sorting preservation #2562
Merged
michaelkain added a commit that referenced this issue Jan 6, 2025
@michaelkain
Merge pull request #2562 from DuckflipXYZ/2491
d7c8af2 
#2491 : HashMap -> LinkedHashMap for sorting preservation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DuckflipXYZ DuckflipXYZ

Labels
backend bug import minor WorkInProgress
Projects
None yet
Milestone
No milestone
2 participants
@michaelkain @DuckflipXYZ