Skip to content

sorting pro badge pins first#4564

Merged
jproberson merged 2 commits intomasterfrom
feat/pro-first
Jan 14, 2026
Merged

sorting pro badge pins first#4564
jproberson merged 2 commits intomasterfrom
feat/pro-first

Conversation

@jproberson
Copy link
Contributor

@jproberson jproberson commented Jan 8, 2026

PR Checklist

  • - Unit and/or e2e tests for the changes that have been added (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Refactoring (no functional changes)
  • Documentation
  • Other:

What is the new behavior?

Primary sort pins by pro then by latest activity

Does this PR introduce a DB Schema Change or Migration?

  • Yes
  • No

Git Issues

Closes #4552

What happens next?

Thank you for the contribution! We will review it ASAP.

If you need more immediate feedback you can reach out to us on Discord in the Community Platform development channel.

@cypress
Copy link

cypress bot commented Jan 8, 2026

onearmy-community-platform    Run #8390

Run Properties:  status check passed Passed #8390  •  git commit 77ca3dc257: Merge branch 'master' into feat/pro-first
Project onearmy-community-platform
Branch Review feat/pro-first
Run status status check passed Passed #8390
Run duration 24m 03s
Commit git commit 77ca3dc257: Merge branch 'master' into feat/pro-first
Committer jproberson
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 1
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 91
View all changes introduced in this branch ↗︎

@jproberson jproberson marked this pull request as ready for review January 8, 2026 03:32
@jproberson jproberson requested a review from a team as a code owner January 8, 2026 03:32
@jproberson jproberson added the Review allow-preview ✅ Has received manual check for malicious code and can be safely built for preview label Jan 8, 2026
Copy link
Member

@benfurber benfurber left a comment

Choose a reason for hiding this comment

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

Looks good! Clear tests, clear logic.

@benfurber benfurber moved this from 🆕 Backlog to In progress in Core Team - BAU list Jan 10, 2026
@jproberson jproberson merged commit e4ad583 into master Jan 14, 2026
12 checks passed
@jproberson jproberson deleted the feat/pro-first branch January 14, 2026 15:19
@github-project-automation github-project-automation bot moved this from In progress to ✅ Done in Core Team - BAU list Jan 14, 2026
@onearmy-bot
Copy link
Collaborator

🎉 This PR is included in version 2.102.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released Review allow-preview ✅ Has received manual check for malicious code and can be safely built for preview

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

[feature request] Show PRO first on Map

4 participants