Skip to content

[imaging_uploader] Sort by UploadDate for most recent uploads #9563

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
Merged
driusan merged 1 commit into from
Feb 18, 2025
Merged

[imaging_uploader] Sort by UploadDate for most recent uploads #9563

driusan merged 1 commit into from
Feb 18, 2025

Conversation

@ridz1208
Copy link
Collaborator

@ridz1208 ridz1208 commented Feb 13, 2025

Brief summary of changes

It seems unlikely that users accessing the imaging uploader are looking for the first 20 scans ever uploaded. It's much more likely that they want the latest...

Testing instructions (if applicable)

Link(s) to related issue(s)

  • Resolves MY OCD ISSUES

@ridz1208 ridz1208 added Category: Feature PR or issue that aims to introduce a new feature Area: UI PR or issue related to the user interface Category: Cleanup PR or issue introducing/requiring at least one clean-up operation Language: SQL PR or issue that update SQL code Beginner Friendly PR or Issue appears to be easy for someone to use to familiarize themselves with LORIS Language: PHP PR or issue that update PHP code Project: C-BIG & ARCHIMEDES Issue or PR related to the C-BIG project Difficulty: Trivial PR or issue whose implementation is obviously correct and no-risk (typo, formatting...) Area: Imaging PR or issue related to imaging Module: imaging_uploader PR or issue related to imaging_uploader module labels Feb 13, 2025
@ridz1208 ridz1208 added this to the 26.0.2 milestone Feb 13, 2025
@driusan driusan modified the milestones: 26.0.2, 26.0.5 Feb 13, 2025
kongtiaowang
kongtiaowang approved these changes Feb 18, 2025
View reviewed changes
Copy link
Contributor

@kongtiaowang kongtiaowang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@driusan driusan merged commit c289820 into aces:26.0-release Feb 18, 2025
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kongtiaowang kongtiaowang kongtiaowang approved these changes

Assignees

@kongtiaowang kongtiaowang

Labels

Area: Imaging PR or issue related to imaging Area: UI PR or issue related to the user interface Beginner Friendly PR or Issue appears to be easy for someone to use to familiarize themselves with LORIS Category: Cleanup PR or issue introducing/requiring at least one clean-up operation Category: Feature PR or issue that aims to introduce a new feature Difficulty: Trivial PR or issue whose implementation is obviously correct and no-risk (typo, formatting...) Language: PHP PR or issue that update PHP code Language: SQL PR or issue that update SQL code Module: imaging_uploader PR or issue related to imaging_uploader module Project: C-BIG & ARCHIMEDES Issue or PR related to the C-BIG project

Projects

None yet

Milestone

26.0.5

Development

Successfully merging this pull request may close these issues.

4 participants

@ridz1208 @kongtiaowang @driusan @nicolasbrossard