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

Update users endpoint to v2 #131

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from
Draft

Update users endpoint to v2 #131

wants to merge 11 commits into from

Conversation

toppercodes
Copy link
Contributor

@toppercodes toppercodes commented Oct 23, 2024

@tothmano tothmano changed the title chore(swagger): Update v2 swagger Update users endpoint to v2 Oct 29, 2024
@tothmano
Copy link
Collaborator

@toppercodes Do you need more help with this? Please see our Slack conversation for reference.

@tothmano
Copy link
Collaborator

@toppercodes Thank you for the update. It looks really good. I've just changed the titles a bit to align with our previous practice.

I'd tested this endpoint before so I didn't check it again this time.

Should we move getCurrentUser to V2 as well? If not, feel free to merge.

@tothmano
Copy link
Collaborator

tothmano commented Nov 4, 2024

@toppercodes Could you please check this disclaimer? 28cc4b1

If you're happy with this, please feel free to merge.

@tothmano
Copy link
Collaborator

tothmano commented Nov 5, 2024

Please hold off merging until final OK from Arne and DX.

@tothmano tothmano marked this pull request as draft November 5, 2024 10:09
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