search

Blamer-io / notifications

GitHub Inbox Watcher
MIT License
1 stars 3 forks source link

Database schema #5

Closed l3r8yJ closed 1 year ago

l3r8yJ commented 1 year ago

@h1alexbel let's discuss here

l3r8yJ commented 1 year ago

@h1alexbel I think it should just be one to many from user to token, wdyt?

h1alexbel commented 1 year ago

@l3r8yJ also, I think it will be handy, I'd we can associate token with alias, like <token> has alias work

Thus, user can simply toggle aliases to get notifications from different tokens

WDYT?

h1alexbel commented 1 year ago

I also think that the notifications system is not related to storing all of this stuff in the database

@l3r8yJ Or I miss something?

l3r8yJ commented 1 year ago

@h1alexbel good suggestion about alias

I was thinking notification the closest service to the github api and it would manage the database, do you have any thoughts on that?

h1alexbel commented 1 year ago

@l3r8yJ we can store data in notifications, but what about hub?

h1alexbel commented 1 year ago

@l3r8yJ I think, as we denote hub as a dispatcher of notifications, this system should be responsible for storing and mapping users and tokens in the database

WDYT?

l3r8yJ commented 1 year ago

@h1alexbel hmm, yeah, that makes sense

h1alexbel commented 1 year ago

@l3r8yJ let's make transition to https://github.com/Blamer-io/hub/issues/24