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.

Sign up for GitHub

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

Jump to bottom

E2415. Reimplement response[s]_controller.rb #92

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

E2415. Reimplement response[s]_controller.rb #92

wants to merge 61 commits into from

Conversation

dlfranks
Copy link

No description provided.

Deana Franks added 30 commits February 27, 2024 23:03
Seed data added
a4580aa
Updated
c7a7ca9
Added minimum required seed data
1641720
Addingadditional seed data
f8851f2
Added many to many relationships for assignment_questionnaire
45c3dae
Updated seed data
b39d354
Added assignment_id to response_map
00fa4df
Seed data went through
b4e5cb8
Added responses service.
4ee0bbc
Added response-dto.rb, response-dto-handler.rb and response-service.rb
1946f65
Added additional columns for the teams table
cff3fad
Created teams_users table
83623f2
Added teams_users table and try to cover all response sub classes. It…
0b88548 
… didn't work.
Added create, edit, and update method
4daf86a
Changed response_service to response_helper, and Added enum_response_map
6c7bb01
Updated the new, edit, show, create, and update method to run
027c0f2
Inatlled active_model_serializers
a691e2d
Removed dto and used Active_Model_serializer instead
17bc708
Updated show, new, create, edit, and update methods
880cbd8
Fixed bugs in new and create methods.
2a3ef7a
Added columns to Participants
3b35c32
Create teams_users table.
752824a
Created sign_up_topic table
850e122
Created signed_up_team table
a0dfcb0
Added associations to TeamsUser, SignUpTopic, and SignedUpTeam
1b6174b
Refactored notify_instructor_on_difference method and relevant methods.
daf8291
Updated questionnaire_by_answer method in response_helper.rb
b2325ef
Added assignment_questionnaire_id to respons_map
84db93a
Updated new and create method and create the locks table
6b887a1
Fixed bugs and updated methods in the Response controller
45b086a
Deana Franks and others added 30 commits March 15, 2024 17:17
Added topic_id to assignment_questionnaires table
34d979c
Fixed bugs and added postman collection
c10414a
Integrated ResponseHandler and Response model.
d3c91b7
Revised the code based on the changes of adding set_content and valid…
97622e9 
…ate method into response model
@vyshnavi-adusumelli
Create response_controller_spec.rb
6e51f4d
Added RSpec tests for response model and controller.
e0a802a
Removed Genfile.lock and seed.rb
3876fd5
Linted code and rolled back associations in response model
7449699
Completed commenting methods
bd6434e
Changed error_msg to response.errors in responses_controller
22c47c6
Removed instance variables in the EmailObject.
1ca2dd4
Changed question to item and fixed bugs.
dbb8ffb
Roll back the swagger schema to the original one and updated gitignore
1362fc6
Cleaned up response_spec.rb
674001d
Uncommented get_questionnairein response_helper
cef2608
Added Responses Swagger endpoints
be33f33
Added Security property into swagger.yaml
4c4d7f5
Fixed swagger.yaml issues and bugs
312117c
Revised response routes and delete response_controller_spec.rb
ca6b869
Updated swagger default host.
5dde80d
Removed create_response_map that was created temperally.
0efd2d2
Split create_update_answers method in create_answers and update_answers
c597633
Refactored validate_params for clarity and modularity broken down int…
efd5b63 
…o several smaller, focused methods, each handling a specific part of the parameter validation process
@jriehle
initial commit for branch
b53ec0e
@jriehle
Merge pull request #2 from maddaicita/refactor/shorten-validate-params
7d3475c 
Refactor/shorten validate params
@jriehle
added show_calibration_results_for_student
6856970
@maddaicita
Merge pull request #4 from maddaicita/refactor/show-calibration-resul…
c4d22cc 
…ts-for-student

Refactor/show calibration results for student
@jriehle
Merge pull request #1 from vyshnavi-adusumelli/main
10320bc 
Test skeletons for E2415
@jriehle
updated response model rspec tests
d2e0dac
@maddaicita
Merge pull request #5 from maddaicita/response_model_rspec
ec6b2a3 
updated response model rspec tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@dlfranks @vyshnavi-adusumelli @jriehle @maddaicita