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

admin-panel: 500 error - data type checks #101

Open
mpede opened this issue May 29, 2023 · 3 comments
Open

admin-panel: 500 error - data type checks #101

mpede opened this issue May 29, 2023 · 3 comments
Labels
admin-panel bug Something isn't working

Comments

@mpede
Copy link

mpede commented May 29, 2023

Hi there,
your work and this bot are much appreciated. I am self-hosting this code, hopefully this feedback helps. I shall open a number of issues - please close/manage at your convenience.

in the admin-panel, if you do not satisfy the correct data type, it will throw a 500 error, but the detailed error msg is only in the logs, the admin user (with no access to logs) will not know whats happening.

A few examples: string instead of integer for health, attack, etc - but also for emoji id - in addition all discord ids are checked to have 17-19 digits. All this will give 500 error in the UI without details.

Furthermore the admin in the UI needs to start over again with entering all values again from scratch (frustrating).

Can these errors not be caught and handled in the code? the 500 screen seems to be native from fastapi or tabler.

Chris

@flaree
Copy link
Member

flaree commented May 29, 2023

Hey,

Yep the admin panel is quite finnicky.
Currently we have no plans to update this as one of our stretch goals is to create a new panel in it's entirety. We are drawing up plans for our future architecture and how it will look in the future.

@mpede
Copy link
Author

mpede commented May 29, 2023

check out an open source tool called directus https://github.com/directus/directus the UI is quite OK

@flaree
Copy link
Member

flaree commented May 29, 2023

Thanks for the suggestion, i'll definitely have a look at it.

I believe our current plans was maybe to use django as a couple of us have experience with it

@laggron42 laggron42 added bug Something isn't working admin-panel labels Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
admin-panel bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants