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.

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

Sort official collections first in API endpoints #40598

Merged
merged 2 commits into from
Mar 25, 2024

Conversation

johnswanson
Copy link
Contributor

For both /api/collection/tree and /api/collection/items, sort collections with official collections coming first.

For both `/api/collection/tree` and `/api/collection/items`, sort
collections with official collections coming first.
@metabase-bot metabase-bot bot added the .Team/AdminWebapp Admin and Webapp team label Mar 25, 2024
@johnswanson johnswanson added the backport Automatically create PR on current release branch on merge label Mar 25, 2024
Copy link

replay-io bot commented Mar 25, 2024

Status Complete ↗︎
Commit 5fedf50
Results
⚠️ 6 Flaky
2375 Passed

@johnswanson johnswanson requested a review from a team March 25, 2024 22:27
test/metabase/api/collection_test.clj Outdated Show resolved Hide resolved
Co-authored-by: Noah Moss <32746338+noahmoss@users.noreply.github.com>
@johnswanson johnswanson enabled auto-merge (squash) March 25, 2024 22:35
@johnswanson johnswanson merged commit dc279fd into master Mar 25, 2024
106 checks passed
@johnswanson johnswanson deleted the sort-official-collections-first branch March 25, 2024 23:26
Copy link

@johnswanson Did you forget to add a milestone to the issue for this PR? When and where should I add a milestone?

@johnswanson johnswanson added this to the 0.49.2 milestone Mar 26, 2024
johnswanson added a commit that referenced this pull request Mar 27, 2024
* Sort official collections first in API endpoints

For both `/api/collection/tree` and `/api/collection/items`, sort
collections with official collections coming first.


Co-authored-by: Noah Moss <32746338+noahmoss@users.noreply.github.com>
johnswanson added a commit that referenced this pull request Mar 27, 2024
* Sort official collections first in API endpoints

For both `/api/collection/tree` and `/api/collection/items`, sort
collections with official collections coming first.

Co-authored-by: John Swanson <john.swanson@metabase.com>
Co-authored-by: Noah Moss <32746338+noahmoss@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Automatically create PR on current release branch on merge .Team/AdminWebapp Admin and Webapp team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants