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

feat: /me endpoint for v2 api #12950

Closed
wants to merge 13 commits into from
Closed

feat: /me endpoint for v2 api #12950

wants to merge 13 commits into from

Conversation

Ryukemeister
Copy link
Contributor

What does this PR do?

Adds a /me endpoint in the v2 api that returns some general info about a user

Copy link

vercel bot commented Dec 27, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ai ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 10, 2024 3:44pm
api ❌ Failed (Inspect) Jan 10, 2024 3:44pm
dev ❌ Failed (Inspect) Jan 10, 2024 3:44pm
4 Ignored Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Jan 10, 2024 3:44pm
cal-demo ⬜️ Ignored (Inspect) Visit Preview Jan 10, 2024 3:44pm
qa ⬜️ Ignored (Inspect) Visit Preview Jan 10, 2024 3:44pm
ui ⬜️ Ignored (Inspect) Visit Preview Jan 10, 2024 3:44pm

Copy link
Contributor

github-actions bot commented Dec 27, 2023

Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link.

async getUser(
@Param("clientId") clientId: string,
@Body() body: GetUserInput,
@GetUser() user: any
Copy link
Contributor

Choose a reason for hiding this comment

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

avoid using any, use the type return by getUser decorator

@PeerRich PeerRich added Medium priority Created by Linear-GitHub Sync api area: API, enterprise API, access token, OAuth platform labels Dec 29, 2023
@Ryukemeister
Copy link
Contributor Author

Closing this since everything here is included in #13006

@Ryukemeister Ryukemeister deleted the me-endpoint-for-v2 branch January 11, 2024 05:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api area: API, enterprise API, access token, OAuth core area: core, team members only Medium priority Created by Linear-GitHub Sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants