Skip to content

Conversation

tobi-bams
Copy link
Collaborator

No description provided.

Copy link
Contributor

@Evanfeenstra Evanfeenstra left a comment

Choose a reason for hiding this comment

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

what is the chat_ids column on the sphinx_graph_suscription exactly for?

I wonder if a table mapping chat_id to subscription_id might be better

@tobi-bams
Copy link
Collaborator Author

what is the chat_ids column on the sphinx_graph_suscription exactly for?

I wonder if a table mapping chat_id to subscription_id might be better

The chat_ids column is used to specify what tribe has access to the particular graph_subscription.

Also, a table mapping chat_id to subscription_id is also a good idea I guess

@Evanfeenstra
Copy link
Contributor

Evanfeenstra commented Jan 17, 2023

The chat_ids column is used to specify what tribe has access to the particular graph_subscription.

Also, a table mapping chat_id to subscription_id is also a good idea I guess

The reason i ask is... Chat used to have a contact_ids row, then we ended up making a ChatMembers table, because you can add other information (role, etc). Also you can do more direct queries without needing to JSON.parse() the row.

For example, the way it is now, how could you get "all subscriptions for a chat"?

@tobi-bams tobi-bams force-pushed the feat/graph-subscription branch from 974c2dd to 88cbd49 Compare January 18, 2023 21:47
@kevkevinpal
Copy link
Contributor

merged in #593

@kevkevinpal kevkevinpal closed this Feb 7, 2023
@tobi-bams tobi-bams mentioned this pull request Feb 7, 2023
@kevkevinpal kevkevinpal reopened this Feb 7, 2023
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.

3 participants