Conversation
tungleduyxyz
commented
Dec 14, 2025
- Kaui 3.0.24 - f873764 : User deletion not working #523
- Product Name and Plan Name Validation on UI #545
- Creating New Catalog Plan Name Validation #544
- Kaui 4.0.1 : Data download #555
- Zero Account/Invoice/Payments count #566
- https://github.com/killbill/technical-support/issues/237
9504f41 to
7c39b51
Compare
There was a problem hiding this comment.
Pull request overview
This PR addresses multiple bug fixes and UI improvements across the Kaui admin interface, focusing on placeholder text corrections, form validation enhancements, and data handling improvements.
- Fixed inconsistent placeholder text across multiple views to accurately reflect the context (Payments, Invoices, Logs, Account Timelines instead of generic "All Accounts")
- Added custom JavaScript form validation for catalog creation to handle jQuery autocomplete interference with HTML5 validation
- Improved error handling and data display for dashboard metrics and invoice filtering
Reviewed changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| test/dummy/config/initializers/money.rb | Sets default currency to USD for Money gem |
| app/views/kaui/payments/_multi_functions_bar.html.erb | Corrects placeholder from "All Accounts" to "All Payments" |
| app/views/kaui/invoices/_multi_functions_bar.html.erb | Corrects placeholder from "All Accounts" to "All Invoices" |
| app/views/kaui/components/dashboard/_card.html.erb | Adds fallback text "20k+" for nil count values |
| app/views/kaui/audit_logs/_multi_functions_bar.html.erb | Corrects placeholder from "All Accounts" to "All Logs" |
| app/views/kaui/admin_tenants/new_catalog.html.erb | Implements custom form validation with error handling for Product and Plan names |
| app/views/kaui/admin_allowed_users/index.html.erb | Refactors delete button implementation and adds error handling for DataTable initialization |
| app/views/kaui/accounts/_billing_details.html.erb | Removes permission check wrapper around "Next invoice date" field |
| app/views/kaui/account_timelines/_multi_functions_bar.html.erb | Corrects placeholder from "All Accounts" to "All Account Timelines" |
| app/helpers/kaui/plugin_helper.rb | Adds route mapping for 'aviate' plugin |
| app/controllers/kaui/invoices_controller.rb | Implements date range filtering for invoice downloads |
| app/controllers/kaui/home_controller.rb | Removes nil fallback ( |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| } | ||
| } %> | ||
| <%= form_tag kaui_engine.admin_allowed_user_path(u.id), method: :delete, style: 'display: inline;', onsubmit: "return confirm('Are you sure?');" do %> | ||
| <%= button_tag type: 'submit', class: 'btn btn-outline-secondary d-inline-flex align-items-center gap-1 kaui-button delete-button custom-hover', style: 'border: none; background: none; padding: 0;' do %> |
There was a problem hiding this comment.
Inline styles override the button's CSS classes, which may cause inconsistent styling. The inline styles 'border: none; background: none; padding: 0;' conflict with the 'btn btn-outline-secondary' classes. Consider moving these styles to a dedicated CSS class or removing conflicting class assignments.
| <%= button_tag type: 'submit', class: 'btn btn-outline-secondary d-inline-flex align-items-center gap-1 kaui-button delete-button custom-hover', style: 'border: none; background: none; padding: 0;' do %> | |
| <%= button_tag type: 'submit', class: 'btn btn-outline-secondary d-inline-flex align-items-center gap-1 kaui-button delete-button custom-hover kaui-btn-no-border' do %> |
7c39b51 to
8afa545
Compare