-
Notifications
You must be signed in to change notification settings - Fork 0
fix(spp_registry): disable gender creation in individual form #30
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
Conversation
Add no_create and no_open options to gender_id field to prevent users from creating or editing gender records inline.
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 enhances data integrity and user experience by restricting the ability to create or edit gender records directly within the individual registrant form. This ensures that gender data is managed through appropriate channels, preventing accidental or unauthorized modifications during individual registration. 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
This pull request aims to disable the creation and editing of gender records from the individual registrant form. The change correctly uses Odoo view options to modify the UI behavior of the gender_id field. My review includes a suggestion to add the no_create_edit option to fully align with the PR's goal and ensure consistency with other fields in the file.
| </group> | ||
| <group> | ||
| <field name="gender_id" readonly="disabled" /> | ||
| <field name="gender_id" readonly="disabled" options="{'no_create': True, 'no_open': True}" /> |
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.
To fully disable creation options as intended, it's best to also include no_create_edit: True. The PR description mentions verifying that 'Create and Edit' is not available, and this option is what controls that feature in the dropdown menu. For consistency with other vocabulary-based fields in this file, such as id_type_id on line 303, which uses {'no_open':True,'no_create_edit':True,'no_create':True}, adding this option would be beneficial.
| <field name="gender_id" readonly="disabled" options="{'no_create': True, 'no_open': True}" /> | |
| <field name="gender_id" readonly="disabled" options="{'no_create': True, 'no_create_edit': True, 'no_open': True}" /> |
Why is this change needed?
Users should not be able to create or edit gender records inline from the individual registrant form.
How was the change implemented?
Added
no_createandno_openoptions to thegender_idfield inindividual_views.xml.New unit tests
Unit tests executed by the author
How to test manually
Related links