feat(spp_hazard): promote to Beta with UX improvements#31
feat(spp_hazard): promote to Beta with UX improvements#31gonzalesedwin1123 merged 2 commits into19.0from
Conversation
…t form view_groups_form_membership inherits from view_individuals_form, so the individual hazard view already covers both individual and group registrants. The separate group view caused duplicate tabs and stat buttons.
Summary of ChangesHello @emjay0921, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request resolves a user interface bug that caused duplicate 'Emergency Response' tabs and 'Impacts' stat buttons to appear on registrant forms. The duplication was a result of an unnecessary Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request resolves a UI bug where the "Emergency Response" tab and "Impacts" stat button were duplicated on registrant forms. The root cause was a redundant Odoo view (view_group_form_hazard) that applied the same UI elements as an existing view due to a chain of view inheritances. The fix correctly removes this redundant view, relying on the view_individual_form_hazard which already handles both individual and group registrants. The change is clean, effective, and improves code maintainability by removing unnecessary code.
Display informative text when no hazard impact records exist instead of showing a blank page.
Why is this change needed?
How was the change implemented?
view_group_form_hazard—view_groups_form_membershipinherits fromview_individuals_form, so one hazard view covers both.readonly="verification_status == 'closed'"to all editable fields in impact form.readonly="status == 'closed'"to all editable fields in incident form.development_statusto Beta, addedemjay0921as maintainer.New unit tests
Unit tests executed by the author
How to test manually
Related links