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

feat: DB queue, error messaging and document upload error codes #1155

Merged
merged 181 commits into from
Nov 23, 2023
Merged

Conversation

jenbutongit
Copy link
Contributor

@jenbutongit jenbutongit commented Nov 22, 2023

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Please also include any acceptance criteria if you have any.

DB based queue and submission process

Architecture decision record
documentation / how to use this feature

Updates to error messages

We've updated error messaging to be in line with GOV.UK design system guidance.See more detail in the pull request #1136

Updates to document upload api

You may now return error codes (alongside status codes) from your document upload API. The form runner will accept the following codes from your document upload API to show tailored errors to the user:

HTTP Status code Error code Tailored error message
422 virusError The selected file for "field_name" contained a virus
422 qualityError The selected file for "field_name" was too blurry

These codes should be sent in the response payload as a string.

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce
the testing if necessary.

  • This is running on a live service! 🥳

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation and versioning
  • I have updated the architecture diagrams as per Contribute.md OR added an architectural decision record entry

ziggy-cyb and others added 30 commits August 2, 2023 12:53
ziggy-cyb and others added 21 commits November 9, 2023 14:38
…rl fields to an empty string before running table modification
…nd dependency resolutions (#1154)

* feat: add more feeOptions - prevent auto submission (#1141)

* add more feeOptions

* fix schema

* change tests for new feeOptions

* custom error message

* renaming to maxAttempts

* Removed extra space

---------

Co-authored-by: Luke Zigler <[email protected]>

* Added README for submitter

* Added docs for enabling the queue service from the runner side

* Updated dependencies

* More dependency updates

* Updated babel dependencies across project

* Updated babel-jest packages

* Added resolution for babel core

* Updated lockfile

* Added resolution for babel traverse

* Allowed postcss GHSA to avoid upgrading webpack

* Fixed md table formatting

* fix: change customPayErrorMessage type to string

---------

Co-authored-by: Luke Zigler <[email protected]>
@jenbutongit jenbutongit changed the title Beta feat: DB queue Nov 22, 2023
@jenbutongit jenbutongit changed the title feat: DB queue feat: DB queue, error messaging and document upload error codes Nov 22, 2023
@jenbutongit jenbutongit merged commit 786d813 into main Nov 23, 2023
22 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.

2 participants