-
Notifications
You must be signed in to change notification settings - Fork 648
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
[FIX] *: fix license typo in manifests t#71407 #302
base: 16.0
Are you sure you want to change the base?
[FIX] *: fix license typo in manifests t#71407 #302
Conversation
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.
Could you submit one commit per module, please?
4b24b2c
to
4dbe699
Compare
@luisg123v Done. |
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.
Fix was lost on the following modules:
custom_list_view
odoo_picking_order_line_views
odoo_sale_order_line_views
p
4dbe699
to
3be8483
Compare
The typo 'licence' was replaced by 'license'.
The typo 'licence' was replaced by 'license'.
The typo 'licence' was replaced by 'license'.
The typo 'licence' was replaced by 'license'.
The typo 'licence' was replaced by 'license'.
The typo 'licence' was replaced by 'license'.
The typo 'licence' was replaced by 'license'.
The typo 'licence' was replaced by 'license'.
The typo 'licence' was replaced by 'license'.
3be8483
to
5f6c9fe
Compare
@luisg123v fixed. |
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 Hi @RisvanaCybrosys Could you merge, please? This eliminates the warnings in odoo.sh |
LGTM @CybroOdoo |
Replace 'licence' with 'license' in manifests that have the typo.