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

Show scheduled jobs in "natural" order #1604

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

francois
Copy link

"Next-job-first" makes more sense for the scheduled tab.

Closes #1580.

@francois
Copy link
Author

Incidentally, I have not been able to find where the default sort order is applied? @bensheldon, feel free to move elsewhere if it makes more sense.

@francois
Copy link
Author

Hold on! This is insufficient. I hadn't tried with multiple pages.

"Next-job-first" makes more sense for the scheduled tab.

Closes bensheldon#1580.
@francois francois force-pushed the sort-scheduled-jobs-in-scheduled_at-order branch from 08a0509 to a049edf Compare February 14, 2025 22:19
@francois
Copy link
Author

All right, I have a solution! I had to pass state to the display_all scope. BatchRecord redefined the display_all scope so I had to introduce an unused parameter to respect the contract.

End of page 1:
Capture d’écran, le 2025-02-14 à 17 06 08

Start of page 2:
Capture d’écran, le 2025-02-14 à 17 06 14

Notice that job 01d... is present on both pages. Not sure if that is acceptable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Inbox
Development

Successfully merging this pull request may close these issues.

[dashboard] Scheduled tasks are shown "backwards"
2 participants