Add Many-to-Many relationship between Group and Member

[0xjei]

Description

This PR introduces a Many-to-Many relationship between Group and Member (a Group can contain multiple Members and a Member can join one or more Groups). The relationship is unidirectional and maintained in the Group. This allows for a join table between Group and Member that does not affect the current APIs, introducing an externally imperceptible (non-breaking) change. This approach was chosen because the join table did not require additional information beyond the Group and Member identifiers having a relationship, denoted by the term Membership.

Related Issue

• fix The same member ID cannot be used in more than 1 group #265
• complete chore: remove primary index from members table #267

Does this introduce a breaking change?

• Yes
• No

[0xjei]

#305 could be potentially related to this. Need further check

[0xjei]

Are we sure we want to maintain this Unexpected await inside a loop no-await-in-loop lint error? @vplasencia

[vplasencia]

Are we sure we want to maintain this Unexpected await inside a loop no-await-in-loop lint error? @vplasencia

Hey! What do you mean? It is better to fix all the errors.

[0xjei]

Are we sure we want to maintain this Unexpected await inside a loop no-await-in-loop lint error? @vplasencia

Hey! What do you mean? It is better to fix all the errors.

Are we sure we want to maintain this Unexpected await inside a loop no-await-in-loop lint error? @vplasencia

Hey! What do you mean? It is better to fix all the errors.

nvm, everything should be okay with the last commit. Used a map with Promise.all() instead of a for loop.

[0xjei]

Are we sure we want to maintain this Unexpected await inside a loop no-await-in-loop lint error? @vplasencia

Hey! What do you mean? It is better to fix all the errors.

Are we sure we want to maintain this Unexpected await inside a loop no-await-in-loop lint error? @vplasencia

Hey! What do you mean? It is better to fix all the errors.

nvm, everything should be okay with the last commit. Used a map with Promise.all() instead of a for loop.

@vplasencia Ready for review :)

[gitpoap-bot]

Congrats, your important contribution to this open-source project has earned you a GitPOAP!

2023 Bandada Contributor:

Head to gitpoap.io & connect your GitHub account to mint!

Learn more about GitPOAPs here.