Skip to content

EDPUB-1246: Fix Special Characters in Email Notifications#60

Merged
DavisGauntUAH merged 2 commits intodevelopfrom
EDPUB-1246-fix-special-char-issues
Apr 26, 2024
Merged

EDPUB-1246: Fix Special Characters in Email Notifications#60
DavisGauntUAH merged 2 commits intodevelopfrom
EDPUB-1246-fix-special-char-issues

Conversation

@camposeddie
Copy link
Copy Markdown
Contributor

@camposeddie camposeddie commented Apr 25, 2024

Description

Display special characters (especially \n, \t, \r, etc. correctly in the email notifications.

Spec

See Ticket: https://bugs.earthdata.nasa.gov/browse/EDPUB-1246

Parallel PR: eosdis-nasa/earthdata-pub-dashboard#39


Validation

  1. Make sure all merge request checks have passed (CI/CD).
  2. Deploy this branch and the API branch EDPUB-1246-fix-special-char-issues to SIT
  3. Create a new request (no need to continue beyond selecting DAAC form.
  4. Sign into another user account (to ensure you receive the email notification).
  5. Navigate to the request conversation.
  6. Send a message with the following text copy/pasted:
Test
	multiline	multitab
	with special's			 characters?
	and \n text? <---
  1. Validate that the message is displayed as expected within the dashboard and within the email notification.
  2. Send a message with the following text copy/pasted:
	Another
test of \r\n\t special character's
and what they \\\ might encode/decode			as!
  1. Validate that the message is displayed as expected within the dashboard and within the email notification.

@DavisGauntUAH DavisGauntUAH merged commit 3e77cca into develop Apr 26, 2024
@DavisGauntUAH DavisGauntUAH deleted the EDPUB-1246-fix-special-char-issues branch April 26, 2024 16:03
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