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

fix: Correct venue sorting to use display name instead of administrat… #414

Merged
merged 1 commit into from
Nov 29, 2024
Merged

fix: Correct venue sorting to use display name instead of administrat… #414

merged 1 commit into from
Nov 29, 2024

Conversation

Lucaci-Andrei
Copy link
Contributor

@Lucaci-Andrei Lucaci-Andrei commented Nov 26, 2024 •
edited
Loading

What

  • Sort venues by display name

How

  • Use venue display name (venue.venueInfo.name) to sort venues alphabetically

ammapspeople
ammapspeople approved these changes Nov 29, 2024
View reviewed changes
Copy link
Contributor

@ammapspeople ammapspeople left a comment

Choose a reason for hiding this comment

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

🎉

@Lucaci-Andrei Lucaci-Andrei changed the base branch from main to release-1.60.0 November 29, 2024 08:35
@Lucaci-Andrei Lucaci-Andrei merged commit 259dc36 into release-1.60.0 Nov 29, 2024
2 checks passed
@Lucaci-Andrei Lucaci-Andrei deleted the bugfix/sort-venues-by-display-name branch November 29, 2024 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ammapspeople ammapspeople ammapspeople approved these changes

Assignees
No one assigned
Labels
patch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Lucaci-Andrei @ammapspeople