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

refactor: UIJSONSerializer dump_list instead of dump_obj #712

Closed

carlinmack wants to merge 0 commits into zenodo:master from carlinmack:master

Contributor

carlinmack commented Feb 9, 2024 •

edited

Loading

Context:

Suggested that this be refactored as it is re-inventing the wheel

Testing:

Checked out a clean version of zenodo-rdm and created a new environment so that it is isolated from any of my local changes.
Made the change and no difference in frontpage rendering
Changed another part of the file to ensure that changes were being reflected in testing
No warnings or alerts in browser console

PS: I have the permissions to merge this

carlinmack closed this

carlinmack force-pushed the master branch from 7a31bb1 to 251aa7d Compare

March 6, 2024 09:01

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

Labels

None yet