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

Group fails to list members if there is more than 100 #779

Closed
llabaj-gj opened this issue Oct 18, 2021 · 0 comments
Closed

Group fails to list members if there is more than 100 #779

llabaj-gj opened this issue Oct 18, 2021 · 0 comments
Assignees
Labels
Bug Something isn't working
Milestone

Comments

@llabaj-gj
Copy link

XP version: 7.3.2 (Did not test this in latest version)

If group has more than 100 members, GraphQL request fails and users are not listed. There is no error shown to the end user, error can be seen only by expecting console log.

What is even worse, if you edit the group and then save changes all members of the group will be removed in the background. I assume this is because group failed to get users and then saves list of members as empty.

Example of failed request:
image

@alansemenov alansemenov self-assigned this Oct 19, 2021
@alansemenov alansemenov added the Bug Something isn't working label Oct 19, 2021
@alansemenov alansemenov added this to the 7.8.0 milestone Oct 19, 2021
@alansemenov alansemenov assigned edloidas and unassigned alansemenov Nov 12, 2021
edloidas added a commit that referenced this issue Nov 16, 2021
Added button to load more members in Group statistics.
All members are loaded if the total number of members are lower than (BATCH_SIZE + 10), otherwise only BATCH_SIZE will be loaded automatically. This is done to prevent a small amount of items to remain unloaded.
Fixed report group visible for non-admin users in groups and roles.
edloidas added a commit that referenced this issue Nov 16, 2021
Added button to load more members in Group statistics.
All members are loaded if the total number of members are lower than (BATCH_SIZE + 10), otherwise only BATCH_SIZE will be loaded automatically. This is done to prevent a small amount of items to remain unloaded.
Fixed report group visible for non-admin users in groups and roles.
alansemenov added a commit that referenced this issue Nov 23, 2021
alansemenov pushed a commit that referenced this issue Nov 23, 2021
Added button to load more members in Group statistics.
All members are loaded if the total number of members are lower than (BATCH_SIZE + 10), otherwise only BATCH_SIZE will be loaded automatically. This is done to prevent a small amount of items to remain unloaded.
Fixed report group visible for non-admin users in groups and roles.
alansemenov added a commit that referenced this issue Nov 23, 2021
alansemenov pushed a commit that referenced this issue Nov 23, 2021
alansemenov pushed a commit that referenced this issue Nov 23, 2021
Added button to load more members in Group statistics.
All members are loaded if the total number of members are lower than (BATCH_SIZE + 10), otherwise only BATCH_SIZE will be loaded automatically. This is done to prevent a small amount of items to remain unloaded.
Fixed report group visible for non-admin users in groups and roles.

(cherry picked from commit e67afd8)
alansemenov added a commit that referenced this issue Nov 23, 2021
(cherry picked from commit f9c7f3d)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants