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

contentUrl for direct download, add section on Converting to Attached RO-Crate #259

Merged
merged 2 commits into from May 23, 2023

Conversation

stain
Copy link
Contributor

@stain stain commented May 17, 2023

New section Converting from Detached to Attached Crate with a suggested algorithm and warnings it may be elaborate. It notes that if the crate is an Attached Crate on the Web then it is easier due to relative paths.

The "Files that are also on the web" section adds contentUrl for the purpose of direct download, while url can be used for landing pages only - as proposed in #189 (comment)

This completes #189 as this section was marked as TODO.

Tagging @jmfernandez @ptsefton @simleo which had views on retrieval in #189.

@stain stain requested a review from ptsefton May 17, 2023 10:48
@stain stain added this to Under discussion/review in RO-Crate specifications via automation May 17, 2023
@stain stain added this to the RO-Crate 1.2 milestone May 17, 2023
@stain stain mentioned this pull request May 18, 2023
29 tasks
@stain stain merged commit 32e56ce into master May 23, 2023
RO-Crate specifications automation moved this from Under discussion/review to Done May 23, 2023
@stain stain deleted the converting-detached-to-attached branch May 23, 2023 12:41
@ptsefton
Copy link
Contributor

I thinnk it's worth revisiting this to see if there is a way to specify where files should be saved when converting from detatched to attached crate -- a "saveAs" relative URI equivalent to @id in an attached crate. We discussed this a long time ago and didn't reach a conclusion.

@ptsefton
Copy link
Contributor

This could be done with an identifier->PropertyValue that specifies the local path

jmfernandez added a commit to inab/WfExS-backend that referenced this pull request Jun 6, 2023
…tentUrl` is used instead of `url` in several contexts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants