GSA / data.gov

Main repository for the data.gov service
https://data.gov
Other
547 stars 87 forks source link

Fix harvest_source notification_emails field #4795

Open btylerburton opened 2 weeks ago

btylerburton commented 2 weeks ago

When creating/editing a harvest source in datagov-harvest-admin, and entering an email into the notification_emails text area, the email is split up into individual characters.

How to reproduce

  1. Create/edit a harvest source in datagov-harvest-admin
  2. Add an email to the Notification_emails field.
  3. Click Submit/Update to save the change

Expected behavior

  1. Email(s) stored as a comma-separated list

Actual behavior

  1. Email addresses are split up as strings and saved as an array.

ex. a@a.com -> ['a', '@', 'a', '.', 'c', 'o', 'm']

Sketch

jbrown-xentity commented 2 days ago

@Jin-Sun-tts mentioned this is annoying, and will take on shortly.