Skip to content

#6883 Force scope on default filters collection#6923

Closed
ngouy wants to merge 1 commit into
activeadmin:masterfrom
ngouy:6883-Scope-default-filter-collections
Closed

#6883 Force scope on default filters collection#6923
ngouy wants to merge 1 commit into
activeadmin:masterfrom
ngouy:6883-Scope-default-filter-collections

Conversation

@ngouy

@ngouy ngouy commented Apr 29, 2021

Copy link
Copy Markdown
Contributor

When we use default filters, or filter of an existing relationship without providing a collection, we enforce the collection to be scope
It is useful when you are using authorization scope to define what resources your current user can access, even in filter values.

@ngouy ngouy changed the title Force scope on default filters collection [#6883] Force scope on default filters collection Apr 29, 2021
@ngouy ngouy changed the title [#6883] Force scope on default filters collection #6883 Force scope on default filters collection Apr 29, 2021
@ngouy

ngouy commented Apr 29, 2021

Copy link
Copy Markdown
Contributor Author

fix #6883

@ngouy ngouy force-pushed the 6883-Scope-default-filter-collections branch from f7a51d2 to 4d1ac17 Compare April 29, 2021 02:07
@ngouy ngouy force-pushed the 6883-Scope-default-filter-collections branch from ce7ef35 to db0e335 Compare August 31, 2021 12:44
@ngouy

ngouy commented May 24, 2022

Copy link
Copy Markdown
Contributor Author

don't really have the bandwidth and not using AA anymore. Feel free to use the code as you wich but I close this

@deivid-rodriguez

Copy link
Copy Markdown
Member

So sorry for not attending this earlier :(. Will try to move this forward at #7511, thanks for your contribution 💪.

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