-
Notifications
You must be signed in to change notification settings - Fork 39
migrate(sdk): projects page api #1224
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
base: main
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Pull Request Test Coverage Report for Build 18971187933Details
💛 - Coveralls |
| select: d => ({ | ||
| groups: d?.groups ?? [], | ||
| groupRoles: (d?.rolePairs ?? []).reduce((acc: Record<string, ProtoRole[]>, gr: ProjectGroupRolePair) => { | ||
| const key = gr.groupId ?? gr.group_id; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am getting a little worries how we are going to get rid of these from codebase, once we are done with migration.
Should we add a TODO?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, these are everywhere. Won't be hard find later.
| } from '~/src'; | ||
|
|
||
| interface useOrganizationProjectsProps { | ||
| showInhreitedProjects?: boolean; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
old typo here, can you take care of this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done. Checked for usage as well.
Uh oh!
There was an error while loading. Please reload this page.