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(series): separate editor for adding series items #680

Merged
merged 17 commits into from
Aug 16, 2021

Conversation

akashgp09
Copy link
Contributor

Summary:
Created separate editor for adding/editing/removing series item.

@coveralls
Copy link

coveralls commented Jul 31, 2021

Coverage Status

Coverage decreased (-0.3%) to 61.148% when pulling dd1e8f9 on akashgp09:series-item into f9a9bba on bookbrainz:series-entity.

Copy link
Contributor

@MonkeyDo MonkeyDo left a comment

Choose a reason for hiding this comment

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

This is looking very nice, well done :)

I think it will make the editing flow much smoother, and we are likely to adapt the same strategy for other entity types in the future (for example Works part of an Edition, with the page attribute on the left-hand side).
This is a great way to refine that workflow with a good use-case, and get feedback from the community.

As I mentioned in one of the comments below I suspect the manual reordering with drag n' drop will be coming in a following PR?

src/server/routes/entity/entity.tsx Outdated Show resolved Hide resolved
src/server/routes/entity/series.js Outdated Show resolved Hide resolved
src/client/entity-editor/series-section/series-editor.tsx Outdated Show resolved Hide resolved
Copy link
Contributor

@MonkeyDo MonkeyDo left a comment

Choose a reason for hiding this comment

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

Good to go ! 🚀

@MonkeyDo MonkeyDo merged commit 5472f93 into metabrainz:series-entity Aug 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants