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.

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

Migrate from Injekt to Kodein for internals to lower chances of crashing #540

Open
4 tasks done
TheBearodactyl opened this issue Dec 9, 2024 · 3 comments
Open
4 tasks done
Labels
enhancement New feature or request

Comments

@TheBearodactyl
Copy link

TheBearodactyl commented Dec 9, 2024

Feature Summary

Would be only internal, but would also lower the amount of crashes due to outdated code, as the current library for dependency injection (Injekt) hasn't been updated in 7 years
seven years

Detailed Description

Self explanatory

Additional Context

Kodein
The current verision of Injekt (specified here)

Submission Checklist

  • I have searched the existing issues and confirm that this feature has not been requested before.
  • I have provided a clear and descriptive title.
  • I am using the latest version of Dantotsu. If not, I have provided a reason for using an older version.
  • I understand that not all feature requests will be accepted, and if declined, I won't resubmit the same request.
@TheBearodactyl TheBearodactyl added the enhancement New feature or request label Dec 9, 2024
Copy link

github-actions bot commented Dec 9, 2024

💡 Thank you for your feature request!
Your request has been successfully submitted and is now under consideration. We value your input in shaping the future of Dantotsu.

📈 What to Expect Next

  • Our team will review your request and assess its feasibility.
  • We may reach out for additional details or clarification.
  • Updates on the request will be provided, and it may be scheduled for future development.

👥 Stay Connected

  • Discord: Join our community to discuss ideas and stay updated.
  • Telegram: Connect with us directly for real-time updates.

We appreciate your suggestion and look forward to potentially implementing it!

@itsmechinmoy
Copy link
Contributor

Why don't you do it?

@TheBearodactyl
Copy link
Author

i tried but the app won't compile lol

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants