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.

Sign up for GitHub

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

Jump to bottom

A few minor fixes #173

Merged

eblondel merged 4 commits into eblondel:master from Rafnuss:master

Feb 5, 2025

Contributor

Rafnuss commented Jan 9, 2025 •

edited

Loading

ExportAs is running for a zenodo draft and returned an html page. Suggesting a fix for checking for is_draft()Note that it actually works on the browser with "? preview=1" but requires authentification.
Seems like there was a typo with uppercase ZENODO vs zenodo.
setSubjects is actually adding subjects rather than starting from an empty list. Suggesting a fix for this.

Rafnuss added 4 commits

January 9, 2025 11:50


          Fix checking for draft

fa36a13


          fix typo

0a5a39b


          Merge branch 'master' of https://github.com/Rafnuss/zen4R

978a42f


          start from empty subject list when using setSubjects

cc70902

Rafnuss changed the title ~~Fix checking for draft~~ A few minor fixes

eblondel merged commit 0f85c25 into eblondel:master

1 check passed

Owner

eblondel commented Feb 5, 2025

Thanks @Rafnuss apologies for the delay, i've completely forgot about your PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet