Refactor how section ordering is handled (#1837) ...

* Extract template customizations into own controller

* Refactor Template model methods

* Fix view logic in draggable sections

* Add button for customising phase

* Fix rubocop violations in GuidanceService

* Rename GuidanceService to GuidancePresenter

* Fix bug in phase preview order sections

* Refactor preview action/controller code

* Fix Section ordering in various views

* Fix rubocop violations

* Fix bug forcing Phases to be modifiable

* Fix broken tests

* Update method to check if section is draggable

* Extract copy action into its own resource

* Extract customization transfers to own controller

* Remove redundant routing condition

* Fix Rubocop violation issues

* Change condition for section draggable

* Refactor instance variable in PlansController

* Fix broken tests

* Add test support for 'de' locale

* Fix merge issue in plans controller

* Fix broken specs in plan#request_feedback

Issue 532 (#1847) ...

* Replaced instances of `raw` with `sanitize`

* Updated Brakeman ignores

* Remove unnecessary sanitation

* Add support for TinyMCE tags

Issue 1845 (#1846) ...

* Fix bug preventing Feedback from being requested

* Fix Rubocop violations

* Extract feedback requests into own controller

* Add test for feedback requests

* Fix Rubocop violations

Exported Plan bug and style fixes: ...

- The csv export now shows the project coversheet text when the option is selected.
- the text export now shows the questions when question text option selected.

Issue/1815 (#1831) ...

* # This is a combination of 2 commits.
# The first commit's message is:

updated logic on number of non-customized templates

# This is the 2nd commit message:

noticed other stats affected by customizations_vs_funder mismatched, so fixed those

* Updated logic on number of non-customized templates, published, unpublished

updated guidance on only display once per question. Added break betw… (#1829) ...

* updated guidance on only display once per question.  Added break between individual guidances

* updated with suggested refactors

Configurable plan access (#1828) ...

* added configuration options to branding.yml for plan access, updated links to be dependant on access

* updated spec to expect multiple calls to branding

Fix TinyMCE JS on forms loaded with DOM (#1827) ...

* Fix TinyMCE JS on forms loaded with DOM

Fixes #1777

* Fix broken specs

* Fix Rubocop style violations

Ignored LineLength violations when string is part of FastGettext

* Updated rubocop style gem

Add tests for locale setting (#1823) ...

* Add tests for locale setting

- Adds Faker::Language module for testing languages with real-ish data
- Refactors global constants into Language model (See Issue#1696)
- Add specs to test locale change works

* Clean up language files to remove long names

* Add DB check in FastGettext initializer

* Set Memory cache for temporary caching solution

* Fix bug in Plan Rule

Merge pull request #1822 from DMPRoadmap/issue-1570 ...

Refactor routes for better maintainability

Merge pull request #1814 from DMPRoadmap/issue-1736 ...

Reset other_organisation when User org changes

Add current version number to application layout (#1826)

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

Ensure annotations destroyed when text is blank ...

When a user updates an existing annotation to set its text to blank, this
should destroy that annotation record on the DB—since they can't be valid without
text.

Remove !bang save and updates when if condition

Refactor template index views to use instance vars

Fix updating questions when annotation text is nil

Deprecate non-idiomatic helper method name

Add specs for editing template question

Reset other_organisation when User org changes ...

- Refactored how user model defines callbacks
- Add specs

Merge pull request #1811 from DMPRoadmap/feature/draggable-tooltip ...

Change tooltip on container page if all modifiable

Merge pull request #1808 from DMPRoadmap/bug/guidance-group-options ...

Fix  duplicates in guidance group options

Change tooltip on container page if all modifiable ...

If all of the sections are modifiable, we don't need the second part of the tooltip that reads: "You may place them before or after the main template sections."

Only show this if some of the Sections are unmodifiable

Fix bug preventing orgs loading on register page ...

Recent updates to the DB have made the `is_other` field on the Orgs table mandatory.
Previously, it was loading null values. This PR updates this to load false values instead.

Fixes #1809

Fix duplicates in guidance group options ...

Restructure the `has_many` association to use SQL's DISTINCT function

Merge branch 'development' into fix/account_creation

Merge branch 'development' into feature/validations

Merge pull request #1803 from DigitalCurationCentre/issue-1738 ...

Issue 1738

Merge pull request #1802 from DMPRoadmap/refactor/database-cleanup ...

Remove redundant database tables and columns

remove comments