-
-
Notifications
You must be signed in to change notification settings - Fork 172
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
Adding an admin panel option when setting up the template #126
Comments
Hello, and thanks for raising an issue. It's a cool idea, but I think it would be hard to implement. Since we can select different ORMs and Databases. Personally I haven't tried a fastapi-admin library, so I don't know if it will work for all possible configurations. If you want to grab this issue, I'm happy to see your pull request. |
hm, thats a really good point. I was thinking along the lines of, the template can have a 'default' option (will have to be an opinionated choice by yourself) for ORM/db. This doesn't mean remove support for others, keep them but have a default option We can then chose to either:
wdyt? It's fine if you think this is just not a feasible feature Also the fastapi-admin has its own ORM implementation with TortoiseORM so that's a limitation. You kind of just need to provide it a db url. But obviously this means, the user may end up having two different ORMs in the project. One for the API and Tortoise for the dashboard The more i think about it, I'm convincing myself out of doing this feature haha |
I had that question in mind. I don't want to restrict people choose ORMs. So we can add this option in features menu and mark it as This feature will work just fine with tortoise ORM, but we cannot provide guarantees about how it's going to work with other ORMs. I guess it's ok to just warn people. |
May be Amis Admin? https://github.com/amisadmin/fastapi_amis_admin |
It looks nice. I'll be ready to add it next week. Or you can submit a PR. On screenshots I've seen only Chinese language. Does it support other languages? |
It support English and have l11n support. But I not found packs for others langs. |
@alldevic, thanks for researching it. English is enough. |
Hey 👋🏽
What are opinions on adding an option to add an admin panel/dashboard which just displays the db tables set up in the template?
The dashboard would be served separately
Thinking about options such as:
I'd be happy to pick this up if it's a wanted feature
The text was updated successfully, but these errors were encountered: