-
-
Notifications
You must be signed in to change notification settings - Fork 85
[17.0][MIG] fleet_vehicle_category: Migration to 17.0 #173
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
[17.0][MIG] fleet_vehicle_category: Migration to 17.0 #173
Conversation
Currently translated at 100.0% (12 of 12 strings) Translation: fleet-14.0/fleet-14.0-fleet_vehicle_category Translate-URL: https://translation.odoo-community.org/projects/fleet-14-0/fleet-14-0-fleet_vehicle_category/pt_BR/
Currently translated at 58.3% (7 of 12 strings) Translation: fleet-14.0/fleet-14.0-fleet_vehicle_category Translate-URL: https://translation.odoo-community.org/projects/fleet-14-0/fleet-14-0-fleet_vehicle_category/tr/
Currently translated at 100.0% (15 of 15 strings) Translation: fleet-14.0/fleet-14.0-fleet_vehicle_category Translate-URL: https://translation.odoo-community.org/projects/fleet-14-0/fleet-14-0-fleet_vehicle_category/it/
Currently translated at 100.0% (15 of 15 strings) Translation: fleet-14.0/fleet-14.0-fleet_vehicle_category Translate-URL: https://translation.odoo-community.org/projects/fleet-14-0/fleet-14-0-fleet_vehicle_category/pt_BR/
Currently translated at 100.0% (16 of 16 strings) Translation: fleet-16.0/fleet-16.0-fleet_vehicle_category Translate-URL: https://translation.odoo-community.org/projects/fleet-16-0/fleet-16-0-fleet_vehicle_category/it/
Currently translated at 100.0% (16 of 16 strings) Translation: fleet-16.0/fleet-16.0-fleet_vehicle_category Translate-URL: https://translation.odoo-community.org/projects/fleet-16-0/fleet-16-0-fleet_vehicle_category/pt_BR/
28bf3f1
to
4634d89
Compare
Hey @OCA/fleet-maintainers would be great if someone could have a look at this module. |
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.
LGTM
This PR has the |
# | ||
msgid "" | ||
msgstr "" | ||
"Project-Id-Version: Odoo Server 16.0\n" |
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.
review version needed
fleet_vehicle_category/i18n/it.po
Outdated
# | ||
msgid "" | ||
msgstr "" | ||
"Project-Id-Version: Odoo Server 14.0\n" |
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.
review version needed
fleet_vehicle_category/i18n/pt_BR.po
Outdated
# | ||
msgid "" | ||
msgstr "" | ||
"Project-Id-Version: Odoo Server 14.0\n" |
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.
review version needed
fleet_vehicle_category/i18n/tr.po
Outdated
# | ||
msgid "" | ||
msgstr "" | ||
"Project-Id-Version: Odoo Server 14.0\n" |
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.
review version needed
|
||
- \[MIG\] Module Migration | ||
|
||
## 16.0.1.0.0 (2024-09-12) |
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.
version 17 history missing
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.
Albeit there are in-code's code documentation related to the code version that needs review (comments made on the code),
LGTM
4634d89
to
4ecdcb6
Compare
@rpsjr , Thanks for suggestions! Please Review. |
all good! |
|
||
|
||
class TestFleetVehicleCategory(TransactionCase): | ||
def setUp(self): |
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, these added unit tests don't really add much value.
There's no business logic in this module, just fields. We don't need to test the ORM
4ecdcb6
to
dacb7c0
Compare
@ivantodorovich, I removed test folder Please Review when you get chance. |
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.
Thanks!
/ocabot merge nobump
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at d8f0cad. Thanks a lot for contributing to OCA. ❤️ |
No description provided.