Skip to content
This repository has been archived by the owner on Dec 2, 2018. It is now read-only.

WIP: streamNotifications: complete rewrite #2

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

brawaru
Copy link
Member

@brawaru brawaru commented Oct 2, 2018

Work In Progress.

This is a rewrite of StreamNotifications module that allows to send notifications about Twitch, Mixer and YouTube streams directly in Discord.


Notification Example
An example of the sent notification for the channel Sarahtje


Rewrite is focused on improving the module and its database structure.
We'll also rewrite the modules used to check for the streams.

  • Plugin
    • Subscribing to channels
    • Unsubscribing from channels
    • Show list of subscriptions
    • Modify Notification Settings
      • By unique ID of Subscription
      • By the Subscription details
    • Modify Guild Settings
  • Module
    • Sending notifications
    • Loading services
    • Sharded Subscription
  • Database
    • Guild Settings
      • DB
        • Create Settings
        • Get Settings
        • Update Settings
        • Delete Settings
      • Controller
        • Get/Set Auto Delete
        • Get/Set Default Channel ID
        • Get/Set Mature Behavior
        • Resolve Channel
        • Resolve Guild
      • Data
        • guildId (string!)
        • autoDelete (boolean?)
        • defaultChannelId (string?)
        • largeAvatars (boolean?)
        • matureBehavior (enum?)
          • Nothing (0)
          • Ignore (1)
          • Banner (2)
          • Governed (3)
    • Notifications Settings
      • DB
        • Create Settings
        • Get Settings
        • Update Settings
        • Delete Settingss
      • Controller
        • Get/Set Auto Delete
        • Get/Set Message Text
        • Get/Set Platform Data
      • Data
        • & <SharedSubscriptionData> (...!)
        • autoDelete (boolean?)
        • messageText (string?)
        • platformData (string?)
        • largeAvatar (boolean?)
    • Notifications
      • DB
        • Save Notification
        • Get Notification
        • Update Notification
        • Delete Notification
      • Controller
        • Get/Set Message ID
        • Get/Set Stream ID
        • Get/Set Payload (Platform Data)
      • Data
        • & <SharedSubscriptionData> (...!)
        • messageId (string!)
        • streamId (string?)
        • platformPayload (string?)
    • Subscriptions
      • DB
        • Get Subscription
        • Get Subscriptions
        • Create Subscription
        • Update Subscription
        • Delete Subscription
      • Controller
        • Is not required.
      • Data
        • guildId (string)
        • alternativeChannel (string?)
        • platform (string)
        • streamerId (string)
        • displayName (string?)
  • Platforms
    • Twitch New · pending revision
    • Mixer · pending revision
    • YouTube · pending revision
    • Goodgame.ru · to be included

@brawaru brawaru added the wip label Oct 2, 2018
@brawaru brawaru mentioned this pull request Oct 7, 2018
17 tasks
@brawaru
Copy link
Member Author

brawaru commented Oct 12, 2018

Added largeAvatars (largeAvatar) boolean notifications setting to control whenever service should or should not include the large avatar of the streamer

@brawaru brawaru added the frozen label Oct 30, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant