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

APP-3213 - add wrappers for org members/invites #157

Merged

Conversation

clintpurser
Copy link
Member

adding some wrappers to the app client for organization settings work.

List Organization Members
Create Organization Invites
Delete Organization Invite
Delete Organization Member

@clintpurser clintpurser requested a review from a team as a code owner December 4, 2023 21:59
@clintpurser clintpurser requested review from stuqdog, purplenicole730 and njooma and removed request for purplenicole730 December 4, 2023 21:59
Copy link
Member

@njooma njooma left a comment

Choose a reason for hiding this comment

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

Can we get some tests please?

@clintpurser clintpurser requested a review from njooma December 6, 2023 16:25
lib/src/app/app.dart Show resolved Hide resolved
lib/src/app/app.dart Outdated Show resolved Hide resolved
Copy link
Member

@njooma njooma left a comment

Choose a reason for hiding this comment

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

Nice

@clintpurser clintpurser requested a review from njooma December 7, 2023 20:54
@clintpurser clintpurser force-pushed the add_app_org_settings_wrappers branch from e3ad4f6 to ffa5375 Compare December 8, 2023 16:49
lib/src/app/permissions.dart Outdated Show resolved Hide resolved
@clintpurser clintpurser merged commit 63add0e into viamrobotics:main Dec 11, 2023
1 check passed
@clintpurser clintpurser deleted the add_app_org_settings_wrappers branch December 11, 2023 15:07
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