Strain duplicates

[draciti]

One strain was extracted (OH12312) for this paper
Tried to add the same strain and see them as duplicates -see screenshot.

Also, should we add WBstrain IDs next to the strains that were extracted?

[valearna]

Show a message in a popup window when the user adds an entity for which there is an entry with name only already in the box.

[valearna]

Added the error message in a popup only for strains - i.e., the popup is not shown for genes and other datatypes

[valearna]

Modify the form to always get the entity added, including the WBStrainID

[valearna]

Can you test the 'add entity' component again @draciti @vanaukenk and let me know if everything looks good?

[vanaukenk]

@valearna
It looks fine to me. Thx.
@draciti - do you also want to double-check?

[draciti]

Will do!

…

________________________________ From: vanaukenk ***@***.***> Sent: Tuesday, September 7, 2021 1:53 PM To: WormBase/author-first-pass ***@***.***> Cc: Raciti, Daniela ***@***.***>; Mention ***@***.***> Subject: Re: [WormBase/author-first-pass] Strain duplicates (#220) @valearna<https://github.com/valearna> It looks fine to me. Thx. @draciti<https://github.com/draciti> - do you also want to double-check? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#220 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AAG4A64H4N33I36TJH2KY2DUAZ3UDANCNFSM5CW5P5IA>. Triage notifications on the go with GitHub Mobile for iOS<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[vanaukenk]

One other thought. If the Strain autocomplete is empty, but the user clicks on 'Add selected', they still get the message that strains in the final list have been replaced.
If the autocomplete field is empty, we should always present the pop-up message about No entities selected.

[valearna]

Thanks @vanaukenk, I'll fix the pop-up messages

[valearna]

@vanaukenk I fixed the pop-up issue

[draciti]

All good, tested both adding new and existing strains, and pop-up functionality. Closing