Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[BDM] - Heartbeat #12

Open
1 of 3 tasks
Peiprjs opened this issue Nov 8, 2021 · 10 comments
Open
1 of 3 tasks

[BDM] - Heartbeat #12

Peiprjs opened this issue Nov 8, 2021 · 10 comments
Labels
2 Well we could do this as soon as we have no other stuff. Blacksheep For anything related to Black Sheep. DomeWork For anything related to Domework. enhancement New feature or request Muser For anything related to Muser.
Milestone
Improvements

Comments

@Peiprjs
Copy link
Member

Peiprjs commented Nov 8, 2021 •
edited by pgamerx
Loading

Is your feature request related to a problem? Please describe.
No

Describe the solution you'd like
Adding a heartbeat function to the bots to detect problems as soon as possible. Maybe make it send a heartbeat per interaction?

Checklist:

  • Black Sheep
  • Muser
  • Domework
@Peiprjs Peiprjs added enhancement New feature or request Blacksheep For anything related to Black Sheep. Muser For anything related to Muser. DomeWork For anything related to Domework. 2 Well we could do this as soon as we have no other stuff. labels Nov 8, 2021
@pgamerx
Copy link

pgamerx commented Nov 9, 2021

Per interaction will be hectic and might just cause us to hit the discord api rate-limit

@Peiprjs
Copy link
Member Author

Peiprjs commented Nov 9, 2021

As far as I know, Cronitor doesn't have ratelimits. Let me read the documentation rq tho

@Peiprjs
Copy link
Member Author

Peiprjs commented Nov 9, 2021

Nah, don't look like it. Could be interesting to have some more data.

@Peiprjs Peiprjs added this to the Improvements milestone Nov 9, 2021
@pgamerx
Copy link

pgamerx commented Nov 9, 2021

Nah, don't look like it. Could be interesting to have some more data.

I said Discord's API

@Peiprjs
Copy link
Member Author

Peiprjs commented Nov 9, 2021

Why would we use DAPI?

@Peiprjs
Copy link
Member Author

Peiprjs commented Nov 9, 2021 •
edited
Loading

We have the CRAPI

@pgamerx
Copy link

pgamerx commented Nov 9, 2021

We have the CRAPI

Makes sense, go ahead.

@pgamerx
Copy link

pgamerx commented Jan 3, 2022

@Peiprjs we have fixed this right?

@Peiprjs
Copy link
Member Author

Peiprjs commented Jan 3, 2022

I just have to implement it, but that's probably a 30 minute job per bot, I can just copy stuff from the other bot.

@pgamerx
Copy link

pgamerx commented Jan 3, 2022

k

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
2 Well we could do this as soon as we have no other stuff. Blacksheep For anything related to Black Sheep. DomeWork For anything related to Domework. enhancement New feature or request Muser For anything related to Muser.
Projects
None yet
Milestone
Improvements
Development

No branches or pull requests
2 participants
@pgamerx @Peiprjs