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

Production < Staging #722

Merged
merged 70 commits into from
Aug 29, 2024
Merged

Production < Staging #722

merged 70 commits into from
Aug 29, 2024

Conversation

Lubosky and others added 30 commits July 15, 2024 21:32
Build support letters if none are present in non-JS version
Signed-off-by: Louis Kirkham <[email protected]>

This PR disables the warning for the CK editor warning. It is a
temporary fix to hide the warning from users while we apply a major
upgrade to CKeditor.
https://app.asana.com/0/1199154381249427/1207754952747949/f
Disables ckeditor version warning
Signed-off-by: Louis Kirkham <[email protected]>

Adds PDF context for letters of support question
https://app.asana.com/0/1199154381249427/1207863714230613/f
Formatting for support letters pdf section
Signed-off-by: Louis Kirkham <[email protected]>

chore: Update form hint for nominee name field

Signed-off-by: Louis Kirkham <[email protected]>
Copy changes to award form
Applies PDF formatting changes to award form
Lubosky and others added 28 commits August 21, 2024 16:54
[KAVS0824] Nomination statistics report
[KAVS0824] Nomination statistics report [QA]
New letters of support question styling
Fix missing value validation on Letters of Support subquestions
Fix: Statistics table QA amendments
…if 2 upload blocks have been filled in and adds an overarching error, we now use individual errors beside each input.

- Hide file upload input after successful upload.
- Fix js & non-js validation for support letter attachments.
- Correct file name styling to govuk-body.
- Update first name label on support letters form.
Letters of support - form UI tweaks
Update copy in submission banner and file upload message
@TheDancingClown TheDancingClown merged commit 817e010 into production Aug 29, 2024
7 of 8 checks passed
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.

6 participants