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

PreprintFile download_url should point to the public url #3852

Merged
merged 2 commits into from
Mar 13, 2024

Conversation

myucekul
Copy link
Contributor

repository_download_file is manager only view whereas repository_file_download is public view. Updating the download url so that correct item url in OAI and JATS metadata for repository items.

repository_download_file is manager only view whereas repository_file_download is public view. Updating the download url so that OAI and JATS metadata contain correct url for repository item.
download_url should point to the public url
@Bbkctp
Copy link
Contributor

Bbkctp commented Dec 19, 2023

Can one of the admins verify this patch?

Copy link
Contributor

@hardyoyo hardyoyo left a comment

Choose a reason for hiding this comment

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

Obviously correct.

@alainna
Copy link
Contributor

alainna commented Mar 13, 2024

@ajrbyers Is there any chance of getting this into -- if not before -- 1.6?
(received another ping about this today -- blocking the indexing of one of our servers :( )

Copy link
Member

@joemull joemull left a comment

Choose a reason for hiding this comment

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

I wonder if it's worth changing the name of the function as well to make it more obviously distinct from repository_download_file?

@ajrbyers
Copy link
Member

I wonder if it's worth changing the name of the function as well to make it more obviously distinct from repository_download_file?

I'ved opened up an issue to look at renaming one or both of the download views for preprint files (#4023).

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.

6 participants