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

Marketplace, Shop, and Affiliate shop owners should be able to add administrators #2187

Closed
spencern opened this issue May 1, 2017 · 2 comments
Assignees

Comments

@spencern
Copy link
Contributor

spencern commented May 1, 2017

Update June 29
This issue is focused on inviting new admins to existing shops.

5 thoughts from a chat @impactmass and I had

  1. This invitation will be sent via the “accounts” dashboard (or the new dashboard that’s created to manage shop users).
  2. An email is generated and sent to the user.
  3. I think the flow might be:
  • Invitation is created
  • New user is generated with existing permissions and with the email specified in the invitation
  • User clicks from email to join -> is logged in via token -> lands on password set/reset
  • User sets new password -> routed to user profile
  1. The invitation form will need a dropdown (I think @rymorgan is working on this/has finished this) to select the correct group to invite the new user to
  2. ‘Appropriate group permissions’ means permissions for the group the user was invited to.
  3. If the user already exists (as a customer or other shop admin), they should be given permissions for the site and sent an email, but an additional user should not be created.

Original Issue
Marketplace, Shop, and Affiliate shop owners should be able to invite people to become administrators.

Newly invited administrators should be granted appropriate permissions based on the marketplace, shop, or affiliate user that invited them. E.g. If a shop owner invites a new admin, that admin should be granted admin permissions for that shop only.

If adding an user or email that is already connected to a Reaction account on this app, we should grant permissions to the existing user.

If we are adding a user or email that does not exist yet, we should grant the new account permissions upon activation.

@spencern spencern added this to the Marketplace Permissions milestone May 1, 2017
@spencern
Copy link
Contributor Author

spencern commented Jun 7, 2017

Seems like a lot of overlap between #2186

@impactmass
Copy link
Contributor

Resolved by #2588

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

No branches or pull requests

3 participants