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

myUserGroups param shows all user groups #1483

Closed
mahalakshme opened this issue Sep 5, 2024 · 4 comments
Closed

myUserGroups param shows all user groups #1483

mahalakshme opened this issue Sep 5, 2024 · 4 comments
Assignees

Comments

@mahalakshme
Copy link
Contributor

mahalakshme commented Sep 5, 2024

Issue:

The logged in user belongs to only 2 user groups. But params.myUserGroups displays all the user groups

Image

Steps to reproduce:

You can try to access the user groups from a rule like below:

Screenshot 2024-09-05 at 10 30 51 AM

Technical reason:

Looks like we are accessing Groups table in realmdb instead of MyGroups from realmdb

@mahalakshme mahalakshme converted this from a draft issue Sep 5, 2024
@mahalakshme mahalakshme moved this from In Analysis to Ready in Avni Product Sep 5, 2024
@himeshr himeshr self-assigned this Sep 5, 2024
@mahalakshme
Copy link
Contributor Author

@himeshr applies to all rules that executes in mobile app. Forgot to add this.

himeshr added a commit that referenced this issue Sep 5, 2024
@himeshr himeshr moved this from In Progress to Code Review Ready in Avni Product Sep 5, 2024
@himeshr
Copy link
Contributor

himeshr commented Sep 5, 2024

After discussion, concluded that we'll not provide any additional info regarding privileges that are granted to the user as part of UserGroups that the user is associated with.
The entity type for params.myUserGroups is now https://github.com/avniproject/avni-models/blob/master/src/MyGroups.js

@1t5j0y 1t5j0y moved this from In Code Review to QA Ready in Avni Product Sep 5, 2024
@himeshr
Copy link
Contributor

himeshr commented Sep 5, 2024

@Dinesh2019
Copy link

  • Getting the below error for all org in the pre-release

org : akrsp
user : dinesh@akrspuat

org : teststatus
user : dinesh@teststatus

Image

@Dinesh2019 Dinesh2019 moved this from QA Ready to QA Failed in Avni Product Sep 6, 2024
@1t5j0y 1t5j0y moved this from QA Failed to QA Ready in Avni Product Sep 6, 2024
@1t5j0y
Copy link
Contributor

1t5j0y commented Sep 6, 2024

Moved back to QA ready as login is working fine on prerelease apk

@Dinesh2019 Dinesh2019 self-assigned this Sep 6, 2024
@Dinesh2019 Dinesh2019 moved this from In QA to Done in Avni Product Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

5 participants