-
Notifications
You must be signed in to change notification settings - Fork 105
Enhance global admin settings design #794
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
base: development
Are you sure you want to change the base?
Enhance global admin settings design #794
Conversation
Reviewer's GuideThis PR refactors the global admin settings UI by introducing a field_groups structure in the form for logical grouping and updating the template to render these groups as Bootstrap tabs and sections, with special handling for payment gateways. It also adds a helper template filter to access form fields dynamically. Class diagram for UpdateSettingsForm and field_groups additionclassDiagram
class GlobalSettingsForm {
+field_groups: list
__init__(...)
}
class UpdateSettingsForm
GlobalSettingsForm <|-- UpdateSettingsForm
File-Level Changes
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @Gagan-Ram - I've reviewed your changes and they look great!
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
@hongquan Some Payment Gateways fields are just rendered as a form field. e.g. the ones which are commented as 'used only in signals'. I wonder what are they really used for, as I could not see any python files using its values. |
The Stripe setup is a bit complicated and maybe confusing:
The same applies to the Paypal plugin. We need to ensure the plugin fields continue to appear in the Global Settings. |
#479
global_admin_design.webm
Summary by Sourcery
Introduce a tabbed layout for global admin settings by grouping fields into logical sections and updating the template to render Bootstrap tabs with separate Stripe and PayPal fieldsets
Enhancements: