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

Added option to sort slides in playlist by "published to" and "status" #254

Merged
merged 10 commits into from
Aug 16, 2024
Merged

Added option to sort slides in playlist by "published to" and "status" #254

merged 10 commits into from
Aug 16, 2024

Conversation

tuj
Copy link
Contributor

@tuj tuj commented Aug 15, 2024 •
edited
Loading

Link to ticket

https://leantime.itkdev.dk/#/tickets/showTicket/1765

Description

  • Changed playlist.slides list columns.
  • Set published.to to now when creating new slides.
  • Added option to sort slides in playlist by published.to and status.

Screenshot of the result

Screenshot 2024-08-15 at 17 05 43 Screenshot 2024-08-15 at 17 06 02

Checklist

  • My code is covered by test cases.
  • My code passes our test (all our tests).
  • My code passes our static analysis suite.
  • My code passes our continuous integration process.

@tuj tuj requested a review from sinejespersen August 15, 2024 15:15
@tuj tuj self-assigned this Aug 15, 2024
@tuj tuj added the enhancement New feature or request label Aug 15, 2024
@tuj tuj marked this pull request as ready for review August 15, 2024 15:16
sinejespersen
sinejespersen previously approved these changes Aug 16, 2024
View reviewed changes
@tuj tuj merged commit 4d5ddd5 into os2display:develop Aug 16, 2024
4 checks passed
@tuj tuj deleted the feature/1765-change-order-by-published branch August 16, 2024 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sinejespersen sinejespersen sinejespersen left review comments

Assignees

@tuj tuj

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tuj @sinejespersen