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

feat(edit) promote edition id partners #8502

Merged

Conversation

davidscotson
Copy link
Collaborator

Closes #8500

feature, change order of edition id dropdown

Technical

Also contains minor fix to move an HTML comment out of a python for loop, as it was getting repeated after every item in the dropdown list.

Testing

Screenshot

Screenshot 2023-11-07 at 19 03 15

Stakeholders

Copy link
Collaborator

@cdrini cdrini left a comment

Choose a reason for hiding this comment

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

Lgtm! One small change

openlibrary/templates/books/edit/edition.html Outdated Show resolved Hide resolved
@cdrini cdrini self-assigned this Nov 7, 2023
@cdrini cdrini added the On testing.openlibrary.org This PR has been deployed to testing.openlibrary.org for testing label Nov 7, 2023
@cdrini
Copy link
Collaborator

cdrini commented Nov 7, 2023

Looks great on testing! Didn't test actually saving an ID but I reckon it should work 👍

@cdrini cdrini merged commit e7b62f5 into internetarchive:master Nov 7, 2023
3 checks passed
@jimchamp jimchamp removed the On testing.openlibrary.org This PR has been deployed to testing.openlibrary.org for testing label Apr 2, 2024
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.

On Edition Edit Page: Promote Trusted Book Providers & Social Catalogues in the Identifier list of Editions
3 participants