Skip to content
This repository has been archived by the owner on Sep 30, 2020. It is now read-only.

Pre-flight checks #164

Open
sr229 opened this issue Feb 20, 2018 · 3 comments
Open

Pre-flight checks #164

sr229 opened this issue Feb 20, 2018 · 3 comments
Assignees
@sr229
Labels
enhancement
Milestone
0.4.4

Comments

@sr229
Copy link
Contributor

sr229 commented Feb 20, 2018

To be able for the bot to run properly, pre-flight checks should be done before init-ing the bot.

Pre-flight checks would check for :

  • Required Tokens
  • Miscellanous tokens, and disable commands that uses those tokens when undefined.
@sr229 sr229 self-assigned this Feb 20, 2018
@sr229 sr229 added this to the 0.4.4 milestone Feb 20, 2018
@Ovyerus Ovyerus self-assigned this Feb 21, 2018
@Ovyerus
Copy link
Contributor

Ovyerus commented Feb 21, 2018 •
edited
Loading

Added in face80f, but only for core settings as the rest are optional(ish),

@Ovyerus Ovyerus closed this as completed Feb 21, 2018
@sr229
Copy link
Contributor Author

sr229 commented Feb 21, 2018

Re-opening as pre-flight should extend loading everything before connecting to Discord

@sr229 sr229 reopened this Feb 21, 2018
@sr229
Copy link
Contributor Author

sr229 commented Feb 21, 2018

Extending it a little :

  • Check connection for redis
  • Validate configuration if it exists
  • lazy-load all events and modules then connect to Discord

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@sr229 sr229

Labels
enhancement
Projects
None yet
Milestone
0.4.4
Development

No branches or pull requests
2 participants
@sr229 @Ovyerus