-
Notifications
You must be signed in to change notification settings - Fork 170
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
Add messages support #700
Comments
@jonocodes Feel free to contribute to this if you want. |
Do you mean "flash" messages from Flask? |
I have not used Flask, but looking at that it does seem to be what I'm talking about. |
I recommend you to check It's very light but depends on |
Thanks. I got this working in my project. Here is what is looks like: I roughly followed this: https://medium.com/@arunksoman5678/fastapi-flash-message-like-flask-f0970605031a Which works with like your middleware suggestion above. And then it uses tabler's alert components. Once I figure out how to get a dev setup in sqladmin I will submit it there. |
Perfect, thanks |
In practice how does one go about developing a feature in the core project? Do I start a demo project locally and link in the local sqladmin libs to see the changes there? Is there a particular demo to work off? |
You can check this: https://github.com/aminalaee/sqladmin/blob/main/CONTRIBUTING.md |
That doc explains how to checkout and build. But where do I actually run something to see my changes? |
You can see the tests. You setup an example application within the project and when you run that example app it shows your changes. |
WIP: #770 |
Checklist
Is your feature related to a problem? Please describe.
I would like to display feedback to users after a form submission, not just error messages. This would allow for warnings and success messages.
Describe the solution you would like.
Django Admin uses this
https://docs.djangoproject.com/en/dev/ref/contrib/messages/#django.contrib.messages.add_message
Describe alternatives you considered
No response
Additional context
I may be willing to work on this, if there is interest.
The text was updated successfully, but these errors were encountered: