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

Extend admin frontend with duplication, copying, and progress operations #215

Merged
merged 14 commits into from
May 1, 2024

Conversation

neketka
Copy link
Collaborator

@neketka neketka commented Apr 30, 2024

Summary

This pull request extends the admin frontend to add more useful features.

  • Added websocket acknowledgements for all gateway functions (to indicate modified ids/success/modified count)
  • Copy events to other organizations
  • Copy achievements to other organizations
  • Copy challenges to other events
  • Add members to organization

Remaining TODOs:

  • View progress screen for looking at event trackers in an event

Test Plan

Test the admin frontend to make sure all these features work correctly.

@neketka neketka changed the title Add acknowledgements Extend admin frontend Apr 30, 2024
@dti-github-bot
Copy link
Member

dti-github-bot commented Apr 30, 2024

[diff-counting] Significant lines: 903.

@neketka neketka changed the title Extend admin frontend Extend admin frontend with duplication, copying, and progress operations Apr 30, 2024
@neketka neketka marked this pull request as ready for review May 1, 2024 04:59
Copy link
Contributor

@cathli66 cathli66 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested all copying on the admin frontend and joining orgs in the backend. There was an issue with CASL permissions (users could not read their auth type), but went ahead and fixed that! Everything looks great :)
LAST BIG PR!!

@cathli66 cathli66 merged commit 22c6fe1 into master May 1, 2024
12 checks passed
@cathli66 cathli66 deleted the nikita/admin-extensions branch May 1, 2024 05:28
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

Successfully merging this pull request may close these issues.

3 participants