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(v2/collab): render viewer-only collaborator modal #3793

Merged
merged 7 commits into from
Apr 28, 2022

Conversation

karrui
Copy link
Contributor

@karrui karrui commented Apr 26, 2022

This PR builds on top of #3788, and is part of the collaborator modal chain.

Problem

Form admin users with only view permissions to a particular form should see a different collaborator modal where they can not update other users' permissions as per design.

This PR adds that modal.

Related to #2487, will close when this entire chain is merged in.

Solution

Features:

  • feat: add ViewOnlyCollaborator modal and its contents.

Before & After Screenshots

New stories have been added for viewer only modal.

@karrui karrui requested a review from timotheeg April 26, 2022 08:20
@karrui karrui force-pushed the form-v2/feat/collab/transfer-ownership-1 branch from a84358b to 79f45f6 Compare April 26, 2022 08:29
@karrui karrui force-pushed the form-v2/feat/collab/view-only-2 branch from ce25775 to b867d8f Compare April 26, 2022 08:29
Base automatically changed from form-v2/feat/collab/transfer-ownership-1 to form-v2/develop April 26, 2022 10:26
@karrui karrui force-pushed the form-v2/feat/collab/view-only-2 branch from b867d8f to 8a46bf7 Compare April 26, 2022 10:27
@karrui karrui marked this pull request as draft April 26, 2022 10:48
@karrui
Copy link
Contributor Author

karrui commented Apr 26, 2022

Realized an opportunity to refactor and reuse components, will open when ready Ready for review

@karrui karrui marked this pull request as ready for review April 27, 2022 04:01
@karrui karrui merged commit fb8cf56 into form-v2/develop Apr 28, 2022
@karrui karrui deleted the form-v2/feat/collab/view-only-2 branch April 28, 2022 10:03
@justynoh justynoh mentioned this pull request Oct 5, 2022
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