Skip to content

Conversation

hasaa-odoo
Copy link

PR for the web framework tutorial,
Note , i forgot to commit the changes for the old section so this will be my first commit (starting from 8th section)

@robodoo
Copy link

robodoo commented Sep 23, 2025

Pull request status dashboard

@cgun-odoo
Copy link

@hasaa-odoo Your PR is from a branch in 18.0 to 19.0, make sure the diff on the pr consists only of your changes

@hasaa-odoo hasaa-odoo changed the base branch from 19.0 to 18.0 September 23, 2025 13:49
@cgun-odoo cgun-odoo removed their request for review September 24, 2025 14:28
@cgun-odoo cgun-odoo self-requested a review September 25, 2025 15:18
Copy link

@cgun-odoo cgun-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't review last file I'll do it tomorrow

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't push anything inside .vscode 💢

openLeads() {
this.action.doAction({
type: 'ir.actions.act_window',
name: 'CRM Leads',

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can make the name translatable
Also check where the name shows up when you run the action. It's important to know

Comment on lines 15 to 17
get Width() {
return 18*this.props.size
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you should maybe name it something like itemWidth

Copy link
Author

@hasaa-odoo hasaa-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just pushed the fixes

@hasaa-odoo hasaa-odoo force-pushed the 18.0-onboarding-owl-hasaa branch from 95411aa to bce32b6 Compare September 30, 2025 12:29
@hasaa-odoo hasaa-odoo force-pushed the 18.0-onboarding-owl-hasaa branch from 11e66f7 to 3a7ac6c Compare September 30, 2025 15:27
@alialfie alialfie removed their request for review October 1, 2025 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants