Skip to content

Conversation

@labkey-klum
Copy link
Contributor

@labkey-klum labkey-klum commented Dec 10, 2024

Related Tickets:

Rationale

This introduces a few new features in the study and publication learn pages:

  • Publication data document ordering. The publication data table will now accept an optional display_order field which defaults to zero. In the event there are multiple publication data rows per publication, the data will be ordered in ascending order.
  • Both studies and publications will now support additional resource links to external data repositories. There is a new table to define external links as well as a mapping table which maps the links to either a study or publication. For study, the links will be rendered in the existing study resources section for study details. For publications, there is a new publication resources component to render the external links for publication details.

image

image

  • A small optimization to the StudyOverview store. This was originally created to decouple the study learn page from the study details page for performance reasons. StudyOverview handles the study details queries and assembly of the data needed to render the details view. This would produce a store with one row per study which seemed a bit wasteful since we are only interested in the single study row we are asking for details about. I added a filter to the cds.metadata.study table to return only the study row of interest,

@github-actions
Copy link

WARNING: This PR appears to have the default title generated by GitHub. Please use something more descriptive.

@labkey-klum labkey-klum changed the title Fb alternate links Study and publication Learn details : External data links Dec 10, 2024
@labkey-klum labkey-klum merged commit acd15a6 into develop Dec 16, 2024
2 of 3 checks passed
@labkey-klum labkey-klum deleted the fb_alternate_links branch December 16, 2024 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants