Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Swap 3290 add tab scientific json metadata #1400

Merged
merged 12 commits into from
Mar 7, 2024

Conversation

yoganandaness
Copy link
Contributor

Description

Added new tab to the Individual Datasets page called as Json Scientific Metadata. This tab will contain the Json form of the Metadata especially only the scientific section ie., .scientificMetadata

Motivation

To show the Scientific Metadata information as a JSON

Fixes:

  • Items added

Changes:

Added new Tab along with other tabs. This tab is configurable based on the configuration settings.

Tests included/Docs Updated?

  • Included for each change/fix?
  • Passing? (Merge will not be approved unless this is checked)
  • Docs updated?
  • New packages used/requires npm install?
  • Toggle added for new features?
  • Requires update of SciCat backend API?

Comment on lines 160 to 161
isLoggedIn &&
isInOwnerGroup,
Copy link
Member

@nitrosx nitrosx Feb 19, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will restrict access to the json scientific metadata only to authenticated users which are also owners of the dataset
It should be fine for now, but we should also open a new issue to address authorization to this tab

@bpedersen2
Copy link
Contributor

probably blocked on #1419

@nitrosx nitrosx merged commit d92641c into master Mar 7, 2024
6 checks passed
@nitrosx nitrosx deleted the SWAP-3290-add-tab-scientific-json-metadata branch March 7, 2024 13:28
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.

4 participants