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

Month End (in janitor.py) #33

Closed
nickalaskreynolds opened this issue Aug 16, 2019 · 2 comments
Closed

Month End (in janitor.py) #33

nickalaskreynolds opened this issue Aug 16, 2019 · 2 comments

Comments

@nickalaskreynolds
Copy link
Contributor

Instead of removing the users from the role, make a duplicate of the role permissions, colour, and name, record all members in that role to a list, remove the old role and insert all the new values. Then make a set of that list and loop through it adding the

This would go from 2 api request per member per role (both reward and colour) to 5 api request per reward + colour role + 1 per member in the reward system. Should be more efficient. I wish there was a better way to mass apply the key roles to people as that is the main slowdown now.

@nickalaskreynolds
Copy link
Contributor Author

need to make an exception for the rules board unless I can think of a better method.

@nickalaskreynolds
Copy link
Contributor Author

This won't work because danbo needs the id. So it would require someone to remove and readd all the roles to danbo. Closing this issue for now.

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

No branches or pull requests

1 participant