-
-
Notifications
You must be signed in to change notification settings - Fork 275
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
Fix(AC): users should be able to make Author Credits optional #871
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working on making this better !
Let me know if you have any question about the comments below
src/client/entity-editor/author-credit-editor/author-credit-section.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is looking pretty good
I tried the PR locally, but it looks like creation an Edition with name section filled, EG selected and AC disabled is not considered valid (submit button is disabled).
Can you please also try this?
Secondly, we should disable the AC edit button when AC is disabled. Currently with AC disabled the AC field is disabled but I can still click edit to open the modal.
Previously it was in the buttonBar section, which is eventually going to be deprecated.
to author-credit-editor actions file, so that it can be shared between the Edition and EditionGroup sections
Reworked mapStateToProps and the new checkbox to disable ACs to fix a couple of issues and simplify a tad
Went from buttonBar to editionSection / editionGroupSection
For new entities whose entitySection is an empty object
which might be undefined (for new entities) in mapStateToProps
Made a mistake when refactoring !
missing `authorCreditEnable: false` for all the edition data
This PR does two things: