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

E2484. Reimplement participants_controller.rb #135

Open
wants to merge 94 commits into
base: main
Choose a base branch
from

Conversation

Pierce57
Copy link

@Pierce57 Pierce57 commented Nov 11, 2024

This PR aims to reimplement the participants_controller.rb and its helper class in the new Expertiza system. This controller manages participants within an assignment, and this reimplementation aims to streamline its methods, eliminate redundancy, and improve code readability and maintainability.

Wiki Page Link: https://wiki.expertiza.ncsu.edu/index.php?title=CSC/ECE_517_Fall_2024_-_E2484._Reimplement_participants_controller.rb

Project Board: https://github.com/users/Pierce57/projects/3

Swagger UI Testing Video: https://www.youtube.com/watch?v=rhP57UexqRs

VCL Environment - Swagger UI: http://152.7.177.32:3002/api-docs/index.html

Pierce57 and others added 30 commits November 11, 2024 13:57
…to 'update_permissions' in 'participants_controller.rb'
…bmit', 'can_review', and 'handle' fields from "participant_params" || Deleting all deprecated Participant routes
Pierce57 and others added 28 commits December 1, 2024 22:19
…' method of ParticipantController for Participant creation
…take_quiz', and 'can_mentor' due to Participant creation approach in the 'add' method
…Split the 'filter_participants' Method into 'filter_user_participants' and 'filter_assignment_participants'
…l edge case, finished rswag tests and tests all passes"
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