Skip to content
This repository has been archived by the owner. It is now read-only.

Retire application and redirect users #46

Merged
merged 1 commit into from Sep 7, 2017
Merged

Retire application and redirect users #46

merged 1 commit into from Sep 7, 2017

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Sep 7, 2017

Part of https://trello.com/c/GtIEFBfC/103-2-retire-govuk-component-guide

@fofr fofr force-pushed the redirect branch from 4061c8e to 9e572cf Sep 7, 2017
@fofr fofr had a problem deploying to govuk-component-guide Sep 7, 2017 Failure
* Redirect all routes to
https://docs.publishing.service.gov.uk/manual/components.html
* Static components now live in a guide at
https://govuk-static.herokuapp.com/component-guide/
* Disable tests which would otherwise fail without routes configured
* Keep changes minimal to keep app in-tact before moving to attic
**Component guides now live within applications and are generated using the [govuk_publishing_components gem](https://github.com/alphagov/govuk_publishing_components). This application is being retired.**

**A list of all component guides is available here: https://docs.publishing.service.gov.uk/manual/components.html. Static components are listed here: https://govuk-static.herokuapp.com/component-guide/**

This comment has been minimized.

@andysellick

andysellick Sep 7, 2017

This is good, but I wonder if it's clear enough - I read the preview of this change and immediately found myself skipping past the bit in bold at the top. Would it be worth moving everything below this into a separate file, linked to by something like 'read the original documentation for this application'?

This comment has been minimized.

@nickcolley

nickcolley Sep 7, 2017
Contributor

Agreed, but do you think we could remove that risk when we move this to the gds-attic?

This comment has been minimized.

@fofr

fofr Sep 7, 2017
Author Contributor

This one-liner was sufficient for specialist-frontend, multipage-frontend and contact-frontend. Which I think is ok given it's going to the attic.

@fofr fofr dismissed andysellick’s stale review Sep 7, 2017

Discussed in person

@nickcolley nickcolley merged commit aad445e into master Sep 7, 2017
3 checks passed
3 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
security/snyk No new issues
Details
@nickcolley nickcolley deleted the redirect branch Sep 7, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.