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

Remove consultation templates redux #3334

Merged
merged 2 commits into from
Jul 10, 2017

Conversation

andrewgarner
Copy link
Contributor

Resurrection of #3006. We now have draft links so we can view draft Consultations in Government Frontend and remove the presentation code from Whitehall.

See also #3067 and Trello

@andrewgarner andrewgarner requested review from gpeng and fofr July 7, 2017 14:41
@andrewgarner andrewgarner force-pushed the remove-consultation-templates-redux branch from bd442e3 to 4f9d5d1 Compare July 7, 2017 14:49
@fofr
Copy link
Contributor

fofr commented Jul 7, 2017

Have you tested locally by creating and previewing a consultation with links? Is it worth putting this on integration to double check?

Otherwise looks good to me, will leave @gpeng to approve.

@andrewgarner
Copy link
Contributor Author

I've not tested locally no, but I'll deploy the branch to integration and do some manual testing.

@andrewgarner andrewgarner changed the title Remove consultation templates redux [DO NOT MERGE] Remove consultation templates redux Jul 7, 2017
@andrewgarner andrewgarner changed the title [DO NOT MERGE] Remove consultation templates redux Remove consultation templates redux Jul 7, 2017
@andrewgarner
Copy link
Contributor Author

Tested successfully on integration by creating a consultation with both a new PDF and a HTML attachment and previewing the draft.

@NickColley
Copy link
Contributor

Epic work @andrewgarner thanks 👍

@andrewgarner andrewgarner merged commit e998dc6 into master Jul 10, 2017
@andrewgarner andrewgarner deleted the remove-consultation-templates-redux branch July 10, 2017 08:57
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

3 participants