Pierre Lamotte (pila-odoo)
Pierre Lamotte (pila-odoo)
Good job mate! 🚀 Can you please squash your commits to only one significant commit?
@Lud0do1202 Can you also please review the name of your PR and your commits titles/messages ? Everything you need to know is [here](https://www.odoo.com/documentation/18.0/contributing/development/git_guidelines.html#commit-message-full-description)
@Lud0do1202 Goo job on commit and PR titles but you are working in the `estate` module. Not the `real_estate` one. 🙄
@Lud0do1202 Congrats for having finished the Python tutorial! 🥳 Can you please check if you considered everything I asked you to do before making Owl fly? To help you get...
@josv-odoo Can you please review the name of your PR and your commits titles/messaged ? Everything you need to know is [here](https://www.odoo.com/documentation/18.0/contributing/development/git_guidelines.html#commit-message-full-description)
@josv-odoo , Please keep this in mind: > @josv-odoo Can you please review the name of your PR and your commits titles/messaged ? Everything you need to know is [here](https://www.odoo.com/documentation/18.0/contributing/development/git_guidelines.html#commit-message-full-description)...
@A-Yehia19 Can you please review the name of your PR and your commits titles/messages ? Everything you need to know is [here](https://www.odoo.com/documentation/18.0/contributing/development/git_guidelines.html#commit-message-full-description)
Hi mate! Can you please explain why you made this PR in the description ? > @A-Yehia19 Can you please review the name of your PR and your commits titles/messages...
As you can see on the runbot, you have some changes requested: • https://runbot.odoo.com/runbot/build/81254345 (issue in your XML) • https://runbot.odoo.com/runbot/build/81254346 (commit titles not following our conventions)
Hi mate! Can you please separate the Owl tutorial in another PR? Keep this one here only for the Python tutorial and squash your commits together.