- 
                Notifications
    
You must be signed in to change notification settings  - Fork 2.7k
 
FRBIN Onboarding #992
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
base: 19.0
Are you sure you want to change the base?
FRBIN Onboarding #992
Conversation
d52fe86    to
    2e5e338      
    Compare
  
    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 ! Here are a few comments :)
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.
As a more general comment take a look at the naming conventions for files and xmlids: https://www.odoo.com/documentation/19.0/contributing/development/coding_guidelines.html
Good work :D
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.
Very good work ! Not much to say.
When you finish this don't forget to do a pass on unit testing :)
| 
           @frbin-odoo Can yoiu please squash everything so that there is only 1 commit per chapter ?  | 
    
5528587    to
    e600ae2      
    Compare
  
    e600ae2    to
    9cecea0      
    Compare
  
    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.
Good overall ! Good job.
Once you corrected everything make sure to squash one commit per chapter :)
8d6c1ec    to
    f89bbbf      
    Compare
  
    
No description provided.