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(EMS-4063): declarations - modern slavery - textarea edge cases #3440

Open
wants to merge 2 commits into
base: main-declarations-modern-slavery
Choose a base branch
from

Conversation

ttbarnes
Copy link
Contributor

Introduction ✏️

This PR updates the textarea mapping functionality so that if any of the following is submitted in the "Modern slavery" textareas, the data will save and render correctly:

  • Pure number.
  • New line characters.
  • Special characters

Resolution ✔️

  • Create E2E tests.
  • Update replaceCharacterCodesWithCharacters and mapTextareaFields to include "Modern slavery" fields.

Miscellaneous ➕

  • Update "cannot skip flow" E2E test to include "Modern slavery" (EMS-4068)

@ttbarnes ttbarnes added enhancement New feature or request feature New functionality tech Technical debt/improvements chore Chore labels Dec 23, 2024
@ttbarnes ttbarnes self-assigned this Dec 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Chore enhancement New feature or request feature New functionality tech Technical debt/improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant