VEuPathDB / EbrcWebsiteCommon

Apache License 2.0
3 stars 3 forks source link

All Studies : release attribute: Bug in release number display for 2 vs 3 releases #197

Open SheenaTomko opened 1 year ago

SheenaTomko commented 1 year ago

We are currently not consistent in which release # and date we display for a study. The same value is shown in both the study summaries table and study details tab, but it may be the original release number or the latest release number, depending on how many versions a study has.

For studies with only 1 version, we display the release # and date correctly. For studies with 2 versions, we display the second release # and date (e.g. Amazonia Brazil shows rel. 21 on 2022-Mar-03, which was the EDA release of the data and not rel.13 which was the original release) For studies with 3 versions, we display the original release # and date (e.g. GEMS1 Case Control shows rel. 5 on 2018-Dec-13 rather than rel. 14 or 21) First we need to decide what should be displayed (most recent version, original version, or a list of all versions?), then we need to implement that.

This is minor and can wait

danicahelb commented 1 year ago

I suggest we always use the original release number. note that this is what the tooltip on the all studies table indicates (see below).

We should also change Release # / date: to Original release # / date: or more simply to just Original release for clarity

image image
dmfalke commented 1 year ago

I think this needs to be handled in the front end code.