-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Fix permissions using dynamic user or role fields #19073
Fix permissions using dynamic user or role fields #19073
Conversation
Dynamic variables were not fetched, so not being parsed and checked
🦋 Changeset detectedLatest commit: bad1f08 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Have tried fixing the unit tests, but have been unable to mock the pinia store successfully. |
Should I just remove the unit tests so it will pass, even though I think it is best to include them? |
I'll look into this later that day 👍 |
Nice, well done @paescuj thanks! Hopefully we can get this released asap? Have been waiting on a fix for almost a month now. |
* Fix permissions using dynamic user or role fields Dynamic variables were not fetched, so not being parsed and checked * Added tests * Pretty * Tried fixing unit tests. * Finish parse-filter test * Finish parse-preset test * Update permissions store test * Update docs for dynamic filters * Add changeset * Update tiny-carpets-remain.md --------- Co-authored-by: Pascal Jufer <pascal-jufer@bluewin.ch> Co-authored-by: Rijk van Zanten <rijkvanzanten@me.com>
Dynamic variables were not fetched, so not being parsed and checked
Fixes #19039, fixes #18898, fixes #19043