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

Refactor routes for better maintainability #1822

Merged
merged 2 commits into from
Aug 20, 2018
Merged

Refactor routes for better maintainability #1822

merged 2 commits into from
Aug 20, 2018

Commits on Aug 17, 2018

  1. Remove unused documentation

    @Bodacious
    Bodacious committed Aug 17, 2018
    Configuration menu
    Copy the full SHA
    7c67e70 View commit details
    Browse the repository at this point in the history

  2. Refactor routes for better maintainability 

    - Remove redundant routes (Fixes #1699, Fixes#1570)
- Remove redundant actions
- Remove redundant policy methods
- Fix invalid actions (public methods defined on controllers)
- Set concern methods as private
- Extract session locale method from ApplicationController into its own controller
- Explicitly define actions in statics controller for better readability
    @Bodacious
    Bodacious committed Aug 17, 2018
    Configuration menu
    Copy the full SHA
    14d1c2a View commit details
    Browse the repository at this point in the history