Skip to content

Conversation

@carstingaxion
Copy link
Collaborator

@carstingaxion carstingaxion commented Aug 13, 2025

Description of the Change

This PR changes the default sorting, that is used for event post listings in the backend. Events are now sorted by event date by default, instead of their publication date.

BEFORE:
image

AFTER:
image

Closes # Part of #694 and directly related to #1134

How to test the Change

Visit https://example.org/wp-admin/edit.php?post_type=gatherpress_event and make sure you see:

  1. the events ordered by event date,
  2. with the oldest first

Changelog Entry

Changed - the default ordering of events in the Admin list view to be ordered by event date, not the published date.

Credits

Props @carstingaxion

Checklist:

  • I agree to follow this project's Code of Conduct.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests pass.

@github-actions
Copy link
Contributor

Preview changes with Playground

You can preview the recent changes for PR#1171 with the following PHP versions:

PHP Version 8.3

PHP Version 7.4

Download .zip with build changes

Made with 💙 from GatherPress & a little bit of WordPress Playground. Changes will not persist between sessions.

@jmarx jmarx self-requested a review August 15, 2025 14:19
Copy link
Collaborator

@jmarx jmarx left a comment

Choose a reason for hiding this comment

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

LGTM

@carstingaxion
Copy link
Collaborator Author

LGTM

Thanks @jmarx!

@mauteri
Copy link
Contributor

mauteri commented Nov 5, 2025

@carstingaxion I see this is approved and looks good, is there a reason to keep this still in draft or is it ready for review and could be merged?

@carstingaxion
Copy link
Collaborator Author

Let me have an additional look, I'm not sure if this is ready.

@carstingaxion
Copy link
Collaborator Author

@carstingaxion I see this is approved and looks good, is there a reason to keep this still in draft or is it ready for review and could be merged?

This is good to be merged.

But, it would only makes sense in combination with #1134, I guess.

@carstingaxion carstingaxion marked this pull request as ready for review December 29, 2025 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants