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

Add sort field to workspace list options #859

Merged
merged 5 commits into from
Feb 26, 2024

Conversation

Maed223
Copy link
Contributor

@Maed223 Maed223 commented Feb 23, 2024

Description

Adds the Sort query parameter to WorkspaceListOptions. Was previously present in AdminWorkspaceListOptions but was missing in WorkspaceListOptions despite the two endpoints having an identical sort behavior, implemented by the same service object class in atlas.

OTHER NOTE: Added a greater degree of specificity to the test covering Sort's created-at behavior for the admin side of things.

Testing plan

Tests covering this Sort behavior have been added to workspace_integration_test.go.

External links

Jira Ticket

Output from tests

Screenshot 2024-02-23 at 11 33 03 AM

@Maed223 Maed223 marked this pull request as ready for review February 23, 2024 18:31
@Maed223 Maed223 requested a review from a team as a code owner February 23, 2024 18:31
@Maed223 Maed223 force-pushed the add-sort-field-workspace-list-options branch from 80718fe to e06fcf8 Compare February 26, 2024 15:46
Copy link
Collaborator

@brandonc brandonc left a comment

Choose a reason for hiding this comment

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

question 👇

workspace_integration_test.go Outdated Show resolved Hide resolved
@Maed223 Maed223 force-pushed the add-sort-field-workspace-list-options branch from e5b9eb8 to c43f037 Compare February 26, 2024 17:20
Copy link
Collaborator

@brandonc brandonc left a comment

Choose a reason for hiding this comment

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

Nice!

@Maed223 Maed223 merged commit 2d5c976 into main Feb 26, 2024
10 checks passed
@Maed223 Maed223 deleted the add-sort-field-workspace-list-options branch February 26, 2024 21:19
Copy link

Reminder to the contributor that merged this PR: if your changes have added important functionality or fixed a relevant bug, open a follow-up PR to update CHANGELOG.md with a note on your changes.

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.

2 participants