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

Add activity log default ordering by occurrence date #2758

Merged
merged 2 commits into from
Jul 10, 2024
Merged

Add activity log default ordering by occurrence date #2758

merged 2 commits into from
Jul 10, 2024

Conversation

nelsonkopliku
Copy link
Member

Description

Adds a default ordering of the activity log by occurrence date, descending: newer occurrences first.

@nelsonkopliku nelsonkopliku self-assigned this Jul 8, 2024
gagandeepb
gagandeepb reviewed Jul 8, 2024
View reviewed changes
Copy link
Contributor

@gagandeepb gagandeepb left a comment

Choose a reason for hiding this comment

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

This implementation would get replaced by the features Flop offers around ordering. But up to you if you need a temporary solution.

@nelsonkopliku
Copy link
Member Author

This implementation would get replaced by the features Flop offers around ordering. But up to you if you need a temporary solution.

Thanks @gagandeepb, you're right. I don't mind if it gets replaced when the time comes 😄

@nelsonkopliku nelsonkopliku force-pushed the activity-log-default-ordering branch from 088f2c9 to 536061e Compare July 10, 2024 06:33
@nelsonkopliku nelsonkopliku merged commit 90a98b5 into main Jul 10, 2024
26 checks passed
@nelsonkopliku nelsonkopliku deleted the activity-log-default-ordering branch July 10, 2024 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@balanza balanza balanza left review comments

@gagandeepb gagandeepb gagandeepb approved these changes

Assignees

@nelsonkopliku nelsonkopliku

Labels
chore
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nelsonkopliku @balanza @gagandeepb