Add markdown declaration to check your answers page, if the form has it#1945
Add markdown declaration to check your answers page, if the form has it#1945thomasiles merged 1 commit intomainfrom
Conversation
If declaration_markdown is present, use it over declaration_text. This will allow us to migrate the existing markdown declarations to markdown.
|
|
🎉 A review copy of this PR has been deployed! It is made of up two components Important Not all of the functionality of forms-runner is present in review apps. You should use the full dev environment to test the functionality which is disabled here. It may take 5 minutes or so for the application to be fully deployed and working. If it still isn't ready For the sign in details and more information, see the review apps wiki page. |



Add markdown declaration to check your answers page, if the form has it
Trello card: https://trello.com/c/b8rdhhe4/2836-support-markdown-in-declaration-field
We are changing the declaration from a text field, rendered with basic formatting, to a markdown field. This matches the "what happens next" and guidance fields, which already use markdown.
The declaration is currently stored in a field in the form and form document as
declaration_text. We will add a new field calleddeclaration_markdownwhich will replace the text version. To support the migration between the two, the runner needs to support both.This PR renders the declaration_markdown if it's present. If it's not, it uses declaration_text as usual.
Things to consider when reviewing