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

Assigning users' permission to a particular filter value #1089

Closed
neeagl opened this issue Sep 9, 2016 · 8 comments
Closed

Assigning users' permission to a particular filter value #1089

neeagl opened this issue Sep 9, 2016 · 8 comments
Labels
enhancement:request Enhancement request submitted by anyone from the community

Comments

@neeagl
Copy link

neeagl commented Sep 9, 2016

To make caravel support multi-tenancy, our data has a unique user id (as a dimension).. we want to be able to give out access to a common database to differnet users. one of the column refers to the user id and only those particular users should be able to access that data in their dashboard.

  1. also is there a way to include a date range filter in the dashboard? right now we have to go to the slice explore view and change it there but its not convenient.
@xrmx
Copy link
Contributor

xrmx commented Sep 11, 2016

The caravel way would be to split your data into multiple tables, one for each user / role. If you use the very same ids as caravel user you may hack something into viz.py query_obj to add a where clause.

@xrmx xrmx added the question label Sep 11, 2016
@neeagl
Copy link
Author

neeagl commented Sep 11, 2016

Thanks xrmx

Also, is there a way we can add time range filter on the dashboard? Currently it looks like we can do that in explorer only, but it's cumbersome to go there every time to change.

If it's not provided today, can you please a way to do that?

@xrmx
Copy link
Contributor

xrmx commented Sep 11, 2016

@neeagl there is already a bug about that. You know, the idea about open source projects is that everyone contribute a bit not that some people work for free for others :)

@xrmx xrmx closed this as completed Sep 11, 2016
@neeagl
Copy link
Author

neeagl commented Sep 11, 2016

I agree :) Just wanted to make sure if there's already a hack around since it's a common use case for several people

Anyway, thanks a lot for your help.

@mistercrunch
Copy link
Member

We may do this at some point

@mistercrunch mistercrunch reopened this Sep 11, 2016
@xrmx
Copy link
Contributor

xrmx commented Sep 12, 2016

@mistercrunch isn't #660 the same?

@mistercrunch mistercrunch added enhancement:request Enhancement request submitted by anyone from the community and removed question labels Sep 12, 2016
@bkyryliuk
Copy link
Member

dup of the #665

thakur00mayank pushed a commit to Altizon/incubator-superset that referenced this issue Oct 27, 2017
Issue: apache#1089

To achieve multi-tenancy:
1. set "ENABLE_MULTI_TENANCY = True" in superset_config file.
2. add column tenant_id String(256) in the tables or views in which you want multi-tenancy.
3. if you are adding the multi-tenancy in existing project then
   make sure that ab_user table have the column tenant_id else alter the table.
4. if you want to enable multi-tenancy with CUSTOM_SECURITY_MANAGER,
   then your custom security manager class should be a subclass of MultiTenantSecurityManager class.

Added the documentation for multi-tenancy.

Fixed few typing errors. Also remove tenant_id from user view.
Fixes few test cases and role update api to support the custom user model.
@ryanblacksa
Copy link

The caravel way would be to split your data into multiple tables, one for each user / role. If you use the very same ids as caravel user you may hack something into viz.py query_obj to add a where clause.

Is there a more concrete solution to this? We need this functionality as well. I see its also requested https://github.com/apache/incubator-superset/issues/660

zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 24, 2021
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 25, 2021
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement:request Enhancement request submitted by anyone from the community
Projects
None yet
Development

No branches or pull requests

5 participants