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.

Sign up for GitHub

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

Jump to bottom

Some permission rules can cause re-sync after every reload of the app #2542

Closed
sleidig opened this issue Aug 27, 2024 · 2 comments · Fixed by #2665
Closed

Some permission rules can cause re-sync after every reload of the app #2542

sleidig opened this issue Aug 27, 2024 · 2 comments · Fixed by #2665
Assignees
@tomwwinter
Labels
released on @master managed by CI (semantic-release) released managed by CI (semantic-release)

Comments

@sleidig
Copy link
Member

sleidig commented Aug 27, 2024 •
edited
Loading

Describe the bug
On dev system the following rule causes the sync to re-start completely every time the app is reloaded or the user logs in anew.

To Reproduce
The following permission rule seems to cause the issue:

{
  "subject": "Child",
  "action": "manage",
  "conditions": {
    "assignedSchoolProject": {
      "$in": "${user.projects}"
    }
  }
}

--> demo.aam-digital.net with user demo (pass)

affects: SSF!, GE?
@github-project-automation github-project-automation bot moved this to Triage / Analysis in All Tasks & Issues Aug 27, 2024
@tomwwinter tomwwinter self-assigned this Nov 15, 2024
@tomwwinter tomwwinter moved this from Todo (ready for work) to In Progress in All Tasks & Issues Nov 15, 2024
@tomwwinter tomwwinter mentioned this issue Nov 15, 2024
Merged
sleidig pushed a commit that referenced this issue Nov 25, 2024
@tomwwinter
fix: prevent re-sync after reloading the app if user.project related …
7115646 
…permission is set (#2665)

fixes #2542 

---------
Co-authored-by: Sebastian Leidig <sebastian@aam-digital.com>
@github-project-automation github-project-automation bot moved this from In Review to Done in All Tasks & Issues Nov 25, 2024
@aam-digital-ci
Copy link
Collaborator

🎉 This issue has been resolved in version 3.43.0-master.8 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@aam-digital-ci aam-digital-ci added the released on @master managed by CI (semantic-release) label Nov 25, 2024
@aam-digital-ci
Copy link
Collaborator

🎉 This issue has been resolved in version 3.43.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@aam-digital-ci aam-digital-ci added the released managed by CI (semantic-release) label Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tomwwinter tomwwinter

Labels
released on @master managed by CI (semantic-release) released managed by CI (semantic-release)
Projects
All Tasks & Issues
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: prevent re-sync after reloading the page if user related permission is set Aam-Digital/ndb-core
3 participants
@sleidig @tomwwinter @aam-digital-ci