search

CDCgov / prime-reportstream

ReportStream is a public intermediary tool for delivery of data between different parts of the healthcare ecosystem.
https://reportstream.cdc.gov
Creative Commons Zero v1.0 Universal
69 stars 39 forks source link

Remove unused catchall mappings #15342

Open JFisk42 opened 1 month ago

JFisk42 commented 1 month ago

User Story

As a ReportStream Developer I want to remove unused mappings so that there aren't a bunch of files hanging around being all confusing and such.

Description/Use Case

During the catchall migration (PR) mappings were moved over without having time to investigate which mappings are not being used currently. There are a number of mappings which were ported over and can safely be deleted. Part of this task will be the investigation to verify which mappings can be deleted without impacting users.

Risks/Impacts/Considerations

If any mappings are deleted which are in use, this would cause errors for any users posting reports that utilize those mappings.

Dev Notes

There are three main folders which are suspected to mostly/entirely contain unused mappings in the catchall/hl7/ path. Datatype, Resources, and Secondary.

Consideration: Many of these mappings files reference other mappings. These will have to be traversed to ensure that all removed mappings are only being referenced by other removed mappings.

Acceptance Criteria

Unused mappings in the prime-reportstream/prime-router/metadata/HL7/catchall/hl7/ directory are deleted.

Andrey-Glazkv commented 1 month ago

Hey team! Please add your planning poker estimate with Zenhub @adegolier @arnejduranovic @brick-green @david-navapbc @jack-h-wang @jalbinson @JFisk42 @mkalish @thetaurean