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

#1907 Refactor response_controller #1414

Merged
merged 18 commits into from Jun 13, 2019

Conversation

Anusha-Godavarthi
Copy link

@Anusha-Godavarthi Anusha-Godavarthi commented Mar 30, 2019

E1907 Refactor response_controller.rb

  • Moved pending_surveys function in the response_controller.rb file to survey_deployment_controller.rb
  • renamed the edit_allowed function to view_allowed in response_controller.rb
  • removed assign_instance_vars section as it was not needed and combined it in edit and new sections
  • removed was_submitted and changed it to prev_submitted in the create method
  • added comments in the private methods
  • Introduced temporary explaining variables in the redirect method
  • Changed the if-else ladder in the redirect method to switch case in response_controller.rb
  • Changed config/routes.rb file to reflect the changes made
  • Made changes in response_controller_spec.rb

@Anusha-Godavarthi Anusha-Godavarthi changed the title Anusha96commits Mastercommits Mar 30, 2019
@expertiza-bot
Copy link
Contributor

1 Message
💬 Thanks for the pull request, and welcome! 🎉 The Expertiza team is excited to review your changes, and you should hear from us soon.

This repository is being automatically checked for code quality issues using Code Climate.
You can see results for this analysis in the PR status below. Newly introduced issues should be fixed before a pull request is considered ready to review.

If you have any questions, please send email to expertiza-support@lists.ncsu.edu.

Generated by expertiza-bot

@expertiza-travisci-bot
Copy link

Hey @Anusha-Godavarthi,
Your changes look good to me! 🎉

@Anusha-Godavarthi Anusha-Godavarthi changed the title Mastercommits #1907 Refactor response_controller Mar 30, 2019
@Winbobob Winbobob added the Merge label May 14, 2019
@efg efg merged commit e8599db into expertiza:beta Jun 13, 2019
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.

None yet

5 participants