Daniel Sass
Daniel Sass
@dependabot rebase
> also does `@Value` overwrite the explicit assignment? seems strange to assign at all? > > **edit** oh also here > > https://github.com/CDCgov/prime-simplereport/blob/5ba4455154efc068c4e5dae201338db95a7402ca/backend/src/main/java/gov/cdc/usds/simplereport/service/TestResultUploadService.java#L116-L117 I do believe `@Value` overwrites the default....
> I also think we should just remove [this line](https://github.com/CDCgov/prime-simplereport/blob/b43dc34051df951e07e1bedb6076e6da5037c397/backend/src/main/java/gov/cdc/usds/simplereport/service/TestResultUploadService.java#L247) since I don't think there's any legit use case for it and it does allow someone (potentially unauthenticated) in a...
Prioritizing this because sample data being deleted becomes significantly more common with the latest data retention work. Suggesting we update the sample data seeding process to look for deleted users...
PII delete doesn't affect API Users, which is the cause of the bug. Patients/the Person table doesn't have a unique constraint that would affect the sample data. BUT - the...
Takeaways from a quick conversation with the team on this: We might need to create an entirely new endpoint to resend as a correction? Will need to touch base with...
> Should it map to something like "User provided SNOMED"? I think the `convertConceptCodeToConceptName` should be updated to default to an empty value rather than a specific description. We're giving...
to clarify: I meant returning and empty string from the function you mentioned. We already have logic in place that only adds a description if we have a value for...
In order to not be blocked by the need for an admin UI, we can do this work just based on the backend.
Is it possible that this has been happening the entire time and the only reason we're seeing it now is because IndexAnalytics team is gone? Possibly related to another okta...