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 (Core | BB-333) Implement a RelationshipType enum #575

Merged
merged 8 commits into from
Mar 22, 2021
Merged

Feat (Core | BB-333) Implement a RelationshipType enum #575

merged 8 commits into from
Mar 22, 2021

Conversation

INNOVATIVEGAMER
Copy link
Contributor

Problem

BB-333
Implement Relationship_Type Enum as per the database (17-03-2021).

Solution

Names in these enum are generated using columns in the 'relationship_type' table as -
source_entity_type + link_phrase + target_entity_type.
But some modifications are made wherever necessary.

Areas of Impact

Added more type perspective for the codebase wherever relationship_type is used.

Implemented Relationship_Type Enum as per the database (17-03-2021).
Names in these enum are generated using columns in the 'relationship_type' table as -
source_entity_type + link_phrase + target_entity_type.
But some modifications are made wherever necessary.
@coveralls
Copy link

Coverage Status

Coverage decreased (-31.7%) to 29.224% when pulling a0104c5 on INNOVATIVEGAMER:IG-Work-2 into 95365c5 on bookbrainz:master.

@coveralls
Copy link

coveralls commented Mar 17, 2021

Coverage Status

Coverage increased (+0.01%) to 60.933% when pulling f4fab17 on INNOVATIVEGAMER:IG-Work-2 into 95365c5 on bookbrainz:master.

@INNOVATIVEGAMER
Copy link
Contributor Author

INNOVATIVEGAMER commented Mar 17, 2021

As per the bookbrainz database (17-03-2021), I exported the relationship_type table in the following file -

relationship_type.xlsx

There are 69 rows in total so created 69 realtionship_types by using format -
source_entity_type + link_phrase + target_entity_type.
but there are changes somewhere , the rows which has some changes are - 13, 16, 17, 19, 41

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.

Nicely done, thanks !
A few nitpicks below, but overall good job!

src/client/entity-editor/relationship-editor/types.ts Outdated Show resolved Hide resolved
src/client/entity-editor/relationship-editor/types.ts Outdated Show resolved Hide resolved
src/client/entity-editor/relationship-editor/types.ts Outdated Show resolved Hide resolved
src/client/entity-editor/relationship-editor/types.ts Outdated Show resolved Hide resolved
src/client/entity-editor/relationship-editor/types.ts 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, thanks ! 🚀

@MonkeyDo MonkeyDo merged commit 0473457 into metabrainz:master Mar 22, 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