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

Multi user switching for quick permisions testing #2056

Open
3 tasks done
JasCodes opened this issue Mar 2, 2022 · 12 comments
Open
3 tasks done

Multi user switching for quick permisions testing #2056

JasCodes opened this issue Mar 2, 2022 · 12 comments
Labels
type:feature New feature or improvement of existing feature

Comments

@JasCodes
Copy link

JasCodes commented Mar 2, 2022

New Feature / Enhancement Checklist

Current Limitation

Even if I try to open another tab, can't have more then one user session.

Feature / Enhancement Description

  • Allows to login multiple users.
  • Allow to logout all users with single click.
  • Easy switching between logged in users.

Example Use Case

Testing permissions

Alternatives / Workarounds

3rd Party References

@parse-github-assistant
Copy link

parse-github-assistant bot commented Mar 2, 2022

Thanks for opening this issue!

  • 🎉 We are excited about your ideas for improvement!

@JasCodes JasCodes changed the title Multi user switching for quick permisions tesing Multi user switching for quick permisions testing Mar 2, 2022
@JasCodes
Copy link
Author

JasCodes commented Mar 2, 2022

@mtrezza feedback please

@davimacedo
Copy link
Member

Have you tried the Browser As User option?

@JasCodes
Copy link
Author

JasCodes commented Mar 3, 2022

As, but it doesn't remember more than one user. And its very cumbersome to switch between multiple users.

@davimacedo
Copy link
Member

I understand your issue, but I am not sure how the UI would work for what you want. Any suggestion? Maybe can you send a mockup?

@JasCodes
Copy link
Author

JasCodes commented Mar 3, 2022

pasre dashboard user browsing mockup

We can probably design a bit better. Also when you refresh the dashboard it auto resets use the master key all the states should be peristed.

@davimacedo Also I am able to edit CLP from the regular users, and thinking from behind the scene dashboard is using master key. Am I right or as I am new to parse, I am getting security wrong?

@davimacedo davimacedo added the type:feature New feature or improvement of existing feature label Mar 3, 2022
@davimacedo
Copy link
Member

Yes. You are right. Parse Dashboard uses the master key. The idea of browse as other multiple users makes sense to me. Would you be willed to send a PR?

@mtrezza
Copy link
Member

mtrezza commented Mar 3, 2022

I think that would certainly be a great further development of the Browse-as-user feature. Excited to see your PR!

@JasCodes
Copy link
Author

JasCodes commented Mar 3, 2022

@mtrezza For sure, but I am really new to parse and unfamiliar to the codebase. Though I will love to be a regular code contributor as started using it with two commercial projects. And its always good to give back. Can somebody from the community who knows the codebase do a quick call with me? Is that possible? Are there any web meetups etc?

@mtrezza
Copy link
Member

mtrezza commented Mar 3, 2022

What you could do is take a look at #1750 that added the feature. The code and the discussion thread should give you an idea of how this feature came to be. Then you could make a suggestion on how you'd extend that and get community feedback.

@ssamuli

This comment was marked as off-topic.

@mtrezza

This comment was marked as off-topic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:feature New feature or improvement of existing feature
Projects
None yet
Development

No branches or pull requests

4 participants