-
Notifications
You must be signed in to change notification settings - Fork 24
[ADD] helpdesk and pos icons #876
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
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 the application's user interface by integrating custom icons for the Helpdesk and Point of Sale modules. These additions aim to improve visual clarity and navigation, making it easier for users to identify and access these specific functionalities within the system. Highlights
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 adds icons for the 'spp_grm' (Helpdesk) and 'point_of_sale' modules. The implementation correctly maps the module names to their menu XML IDs and new icon paths. My feedback focuses on improving the consistency of the icon filenames for better long-term maintainability. I've suggested renaming the new icon files to follow the kebab-case convention used elsewhere in the project.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## 17.0 #876 +/- ##
==========================================
- Coverage 81.22% 81.22% -0.01%
==========================================
Files 818 818
Lines 24956 24966 +10
Branches 2917 2917
==========================================
+ Hits 20270 20278 +8
Misses 3953 3953
- Partials 733 735 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
- Renamed all icon-* files to openspp-icons-menu-* for consistency
- Updated icon references in ir_module_module.py ICON_MAP
- Follows kebab-case naming convention throughout
- All menu icons now use the standard openspp-icons-menu-{menuname} pattern
- Updated test_ir_module_module.py to use new openspp-icons-menu-discuss.png - Aligns with standardized openspp-icons-menu-* naming pattern - Prevents test failure after icon renaming refactor
|



Why is this change needed?
How was the change implemented?
New unit tests
Unit tests executed by the author
How to test manually
Related links
Note
Updates menu icon mappings to new assets (including helpdesk and POS) and adds tests for phone validation activate/deactivate plus updated icon test.
spp_base_common/models/ir_module_module.py):openspp-icons-*assets.spp_grm(helpdesk) andpoint_of_salewith corresponding icons.tests/test_ir_module_module.py: Update expectedmail.menu_root_discussicon to new path.tests/test_phone_number_validation.py: Addtest_06_activate_phone_validationandtest_07_deactivate_phone_validationto verify state transitions.CHANGELOG.mdwith added phone validation state tests.Written by Cursor Bugbot for commit 31e2f0b. This will update automatically on new commits. Configure here.