-
-
Notifications
You must be signed in to change notification settings - Fork 645
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
Django API Support: Defcon cog #116
Comments
Comment from Johannes Christ: marked this issue as related to #87 |
Comment from Johannes Christ: marked this issue as related to #78 |
Comment from Johannes Christ: marked this issue as related to #86 |
Comment from Johannes Christ: marked this issue as related to #85 |
Comment from Johannes Christ: marked this issue as related to #84 |
Comment from Johannes Christ: marked this issue as related to #83 |
Comment from Johannes Christ: marked this issue as related to #82 |
Comment from Johannes Christ: marked this issue as related to #81 |
Comment from Johannes Christ: marked this issue as related to #80 |
Comment from Johannes Christ: We need to think about a smart way to store this kind of things in the bot. Maybe we should have a CREATE TABLE bot_settings (
category VARCHAR(20),
settings JSONB
); |
Comment from Johannes Christ: The issue with that approach would be that we wouldn't be able to properly validate settings unless we would hardcode the per-category setting format. |
Comment from Johannes Christ: ... which might not be a horrible idea. |
This was implemented in 2469592 |
Originally posted by Johannes Christ:
Add support for using the Defcon cog with the new Django API. The branch should be branched off https://gitlab.com/python-discord/projects/bot/merge_requests/67 and the merge request should head back into the linked branch.
This is a good first issue - if you have any questions about this or want more information, don't hesitate to ask!
The text was updated successfully, but these errors were encountered: