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

Snapshots #1295

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Snapshots #1295

wants to merge 6 commits into from

Conversation

fherreazcue
Copy link
Collaborator

Added custom title and description to snapshots, and the option to edit them as long as they don't have a DOI assigned.

@fherreazcue fherreazcue linked an issue Dec 9, 2022 that may be closed by this pull request
@stuzart
Copy link
Member

stuzart commented Dec 13, 2022

Paused until the new year, need to look more closely at how metadata is updated and passed

@stuzart stuzart closed this Dec 13, 2022
@fherreazcue
Copy link
Collaborator Author

Merged main to branch, updated tests and resolved conflicts.

@fherreazcue fherreazcue reopened this Aug 23, 2023
@fbacall
Copy link
Contributor

fbacall commented Aug 23, 2023

Is this for 1.14 or 1.15?

@fbacall fbacall changed the base branch from seek-1.13 to seek-1.14 August 23, 2023 12:13
@fbacall fbacall changed the base branch from seek-1.14 to main August 23, 2023 12:15
@stuzart
Copy link
Member

stuzart commented Aug 23, 2023

Is this for 1.14 or 1.15?

1.15

@@ -43,6 +43,7 @@ def export_to_zenodo(access_token, extra_metadata = {})
metadata.merge!(extra_metadata.to_h.deep_symbolize_keys)

#FIXME: this is a quick hack
metadata[:title] = 'not set' if metadata[:title].blank?
Copy link
Contributor

Choose a reason for hiding this comment

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

The Zenodo export needs thinking about, because the snapshot title/description may not be what you want to be the title/description of the Zenodo entry (currently it would be the title/description of the resource being snapshotted).

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.

need for snapshot's title/description
3 participants