THE-BWC / DiscordOperationsBot

MIT License
0 stars 0 forks source link

30 Minutes notification message #30

Closed Goltred closed 8 months ago

Goltred commented 8 months ago

This PR includes the logic related to sending the 30 minutes notification message on configured channels. This includes:

@PatrickPedersen - I'm going to need some help completing those models that we need to get user and game information since i don't know the structure of the xenforo db :)

thebwc-sonarqube[bot] commented 8 months ago

Failed

Analysis Details

53 Issues

Coverage and Duplications

Project ID: THE-BWC_DiscordOperationsBot_AY3xt-Bgmgcb4U3BetPR

View in SonarQube

PatrickPedersen commented 8 months ago

@Goltred If you look at these queries, you should be able to get some, if not all of the way there: https://github.com/THE-BWC/Discord_OpServBot/blob/main/models/xenforo/xenProvider.js

Otherwise, I'll look at it later when I have more time.

Goltred commented 8 months ago

Yeah, no worries there's no pressure for this.

The queries do help but we might still run into issues it peewee doesn't like that I might be missing some fields in the model.

thebwc-sonarqube[bot] commented 8 months ago

Failed

Analysis Details

53 Issues

Coverage and Duplications

Project ID: THE-BWC_DiscordOperationsBot_AY3xt-Bgmgcb4U3BetPR

View in SonarQube

thebwc-sonarqube[bot] commented 8 months ago

Failed

Analysis Details

53 Issues

Coverage and Duplications

Project ID: THE-BWC_DiscordOperationsBot_AY3xt-Bgmgcb4U3BetPR

View in SonarQube

thebwc-sonarqube[bot] commented 8 months ago

Failed

Analysis Details

42 Issues

Coverage and Duplications

Project ID: THE-BWC_DiscordOperationsBot_AY3xt-Bgmgcb4U3BetPR

View in SonarQube

thebwc-sonarqube[bot] commented 8 months ago

Failed

Analysis Details

29 Issues

Coverage and Duplications

Project ID: THE-BWC_DiscordOperationsBot_AY3xt-Bgmgcb4U3BetPR

View in SonarQube

thebwc-sonarqube[bot] commented 8 months ago

Failed

Analysis Details

29 Issues

Coverage and Duplications

Project ID: THE-BWC_DiscordOperationsBot_AY3xt-Bgmgcb4U3BetPR

View in SonarQube

thebwc-sonarqube[bot] commented 8 months ago

Failed

Analysis Details

7 Issues

Coverage and Duplications

Project ID: THE-BWC_DiscordOperationsBot_AY3xt-Bgmgcb4U3BetPR

View in SonarQube

thebwc-sonarqube[bot] commented 8 months ago

Failed

Analysis Details

3 Issues

Coverage and Duplications

Project ID: THE-BWC_DiscordOperationsBot_AY3xt-Bgmgcb4U3BetPR

View in SonarQube

thebwc-sonarqube[bot] commented 8 months ago

Failed

Analysis Details

3 Issues

Coverage and Duplications

Project ID: THE-BWC_DiscordOperationsBot_AY3xt-Bgmgcb4U3BetPR

View in SonarQube

thebwc-sonarqube[bot] commented 8 months ago

Failed

Analysis Details

2 Issues

Coverage and Duplications

Project ID: THE-BWC_DiscordOperationsBot_AY3xt-Bgmgcb4U3BetPR

View in SonarQube

thebwc-sonarqube[bot] commented 8 months ago

Failed

Analysis Details

0 Issues

Coverage and Duplications

Project ID: THE-BWC_DiscordOperationsBot_AY3xt-Bgmgcb4U3BetPR

View in SonarQube

thebwc-sonarqube[bot] commented 8 months ago

Passed

Analysis Details

0 Issues

Coverage and Duplications

Project ID: THE-BWC_DiscordOperationsBot_AY3xt-Bgmgcb4U3BetPR

View in SonarQube

PatrickPedersen commented 8 months ago

@Goltred Sorry for all the pings. Technically this is now ready to merge with main. Is there anything else you want to do on this PR? Or should we just merge?

Goltred commented 8 months ago

Sorry, i think it should be good to go. I'm not merging without consent though so feel free to do so? :D