Skip to content

Conversation

@lavanya3k
Copy link
Collaborator

This is a pull request to update the Earthdata support email address in the error message. The email address has been updated to '[email protected]'.

Code changes: The following changes are made in the check_messages_override.json and ruleset.json. In the check_messages_override.json, the email address was updated in the 13 different schema checks. These include

  • science_keywords_gcmd_check
  • location_gcmd_check
  • organization_short_name_gcmd_check
  • organization_long_name_gcmd_check
  • instrument_short_name_gcmd_check
  • instrument_long_name_gcmd_check
  • platform_short_name_gcmd_check
  • data_format_gcmd_check
  • platform_long_name_gcmd_check
  • spatial_keyword_gcmd_check
  • platform_type_gcmd_check
  • campaign_short_name_gcmd_check
  • campaign_long_name_gcmd_check

In the ruleset.json, the email address is updated in Instrument Long Name Check and Instrument Short Name Check.

Reproduce the result:
C1537476328-CDDIS --format umm-c
C1605632902-CDDIS --format umm-c

Expected outcome:
Screenshot 2025-05-19 at 1 43 07 PM

@bhawana11
Copy link
Collaborator

The following concept IDs were tested, and the email address was correctly updated in the error message:
C1000000010-CDDIS
C2768953472-CDDIS
C1605632902-CDDIS
C1537476328-CDDIS
C1000000042-CDDIS
C1627523804-LARC

@slesaad slesaad merged commit 20fb606 into dev Aug 8, 2025
1 check passed
@slesaad slesaad deleted the earthdata-email-la branch August 8, 2025 19:25
@lavanya3k
Copy link
Collaborator Author

Testing in the "dev" branch: The following collection IDs were tested and passed the rule check for Earthdata email.
Screenshot 2025-09-15 at 1 08 50 PM

@lavanya3k
Copy link
Collaborator Author

Tested in the dev branch, and the code changes work.

Example:
C1537476328-CDDIS --format umm-c
C1000000010-CDDIS --format umm-c
C1000000015-CDDIS --format umm-c

Expected output:
Screenshot 2025-10-14 at 12 45 46 PM

@lavanya3k lavanya3k mentioned this pull request Oct 17, 2025
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.

4 participants