show individual name rather than uuid in bulk import listing#1249
show individual name rather than uuid in bulk import listing#1249JasonWildMe merged 2 commits intomainfrom
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1249 +/- ##
==========================================
- Coverage 8.59% 8.59% -0.01%
Complexity 939 939
==========================================
Files 623 623
Lines 67991 67993 +2
Branches 12492 12493 +1
==========================================
Hits 5845 5845
- Misses 61548 61550 +2
Partials 598 598
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| @@ -115,6 +115,7 @@ const BulkImportTask = observer(() => { | |||
| user: item.submitter?.displayName || "-", | |||
| occurrenceID: item.occurrenceId || "-", | |||
| individualID: item.individualId || "-", | |||
There was a problem hiding this comment.
You mean to keep individualID here and add individualName? Or replace individualID with individualName?
There was a problem hiding this comment.
As a double check: is there the possibility for more confusion here?
Could the user enter in their spreadsheet:
"Bob"
our logic reconciles "Bob" to individual <some_UUID>
we then return that individual's display name, which is a complex set of logic that might return some other value like "MX-4567"?
Is it worth just showing the Excel field value "Bob" so long as it can be validated to an existing individual's UUID?
individualDisplayNameto each encounter in api for bulk import tasksPR fixes #1248