Conversation
| <ExportMetadataDropdown | ||
| datasetPersistentId={datasetPersistentId} | ||
| datasetVersion={datasetVersion} | ||
| canUpdateDataset={permissions.canEditOwnerDataset} |
There was a problem hiding this comment.
I hope this is the right place to put this comment but @ChengShi-1 and I just discussed how the Member role (which is what Preview URLs use) should be enough to export the various metadata formats.
There was a problem hiding this comment.
Thanks for catching this@pdurbin , I just finish the fix
There was a problem hiding this comment.
@ChengShi-1 oh, above can you please update "Also, for a draft version, as a dataset owner, the button should show and return draft version metadata"?
There was a problem hiding this comment.
Sure, just updated!
Co-authored-by: Philip Durbin <philip_durbin@harvard.edu>
pdurbin
left a comment
There was a problem hiding this comment.
The code looks good. Tests are passing. I tested the functionality. I'm happy to merge this myself, if that's agreeable to everyone.

What this PR does / why we need it:
Which issue(s) this PR closes:
Special notes for your reviewer:
datasetIsReleased,datasetIsDeaccessionedparam fromExportMetadataDropdowncomponent, usinguseExportMetadatainstead to handle showing/hiding the "Export Metadata" buttonSuggestions on how to test this:
Does this PR introduce a user interface change? If mockups are available, please link/include them here:
Is there a release notes or changelog update needed for this change?:
yes
Additional documentation: