hubmapconsortium / portal-ui

HuBMAP Data Portal front end
https://portal.hubmapconsortium.org
MIT License
12 stars 2 forks source link

NickAkhmetov/CAT-949 Remove provenance section, convert donor page to TypeScript #3571

Closed NickAkhmetov closed 1 month ago

NickAkhmetov commented 1 month ago

Summary

After investigations related to CAT-751, Zhou and Bill have determined there is no longer any provenance information to display for donors. Historically, there were lab nodes which were showing up but did not have any related information, so they removed them approximately 4 months ago. As a result, the provenance graphs no longer had any valid relationships to display, so they crashed; therefore, removing this section in favor of the derived entities section removes an unnecessary visual component.

I have also converted the Donor detail page to TypeScript, resolved any resulting reported type errors, and updated the Protocols component to use the withShouldDisplay HOC.

Design Documentation/Original Tickets

https://hms-dbmi.atlassian.net/browse/CAT-751 https://hms-dbmi.atlassian.net/browse/CAT-949

Testing

I A/B tested this donor page: https://portal.hubmapconsortium.org/browse/donor/a0bdb4249eb3a6bd1865f948875c2f96 on production and local.

Screenshots/Video

Production screenshot (Note that prod is also missing organ data in the screenshot, this was fixed in #3567)

![image](https://github.com/user-attachments/assets/693bb9c1-283e-48d9-aec6-4beff09a6643)

Fixed screenshot

![image](https://github.com/user-attachments/assets/5e682d8f-2be3-4372-b253-ecf1bc80fc37)

Checklist

Additional Notes

I think we can probably do a lot more with the Summary section of donor pages. Right now the information there is very minimal - I think we should surface some useful information so people can learn more about the donor at a glance, e.g. demographic info, organs that were analyzed from this donor, etc.