Skip to content
This repository has been archived by the owner on Jan 21, 2024. It is now read-only.

Dependency Injection #29

Open
MitchTalmadge opened this issue Jul 2, 2020 · 1 comment
Open

Dependency Injection #29

MitchTalmadge opened this issue Jul 2, 2020 · 1 comment
Labels
Feature New feature or request Priority-2 Mid Priority

Comments

@MitchTalmadge
Copy link
Owner

Shouldn't be using static classes for everything. https://github.com/inversify/InversifyJS
@MitchTalmadge MitchTalmadge added Feature New feature or request Priority-2 Mid Priority labels Jul 2, 2020
@MitchTalmadge
Copy link
Owner Author

I've started making new classes so that they are non-static. Once I get time I'd like to go through and convert most if not all static classes (definitely all static services) to non-static. Pass in guildContext into the constructor instead of every single method.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Feature New feature or request Priority-2 Mid Priority
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@MitchTalmadge