Skip to content

Conversation

@dinakdugar
Copy link
Contributor

@dinakdugar dinakdugar commented Feb 19, 2025

Implementing feedback from SME: #12001 (review)

Target: 18.0+

Task: https://www.odoo.com/odoo/project/3835/tasks/4590847

@robodoo
Copy link
Collaborator

robodoo commented Feb 19, 2025

Pull request status dashboard

@dinakdugar dinakdugar self-assigned this Feb 19, 2025
@dinakdugar dinakdugar added the 2 label Feb 19, 2025
@dinakdugar dinakdugar changed the title [FIX] sales: gelato feedback [IMP] sales: gelato feedback Feb 19, 2025
@C3POdoo C3POdoo requested a review from a team February 19, 2025 18:15
@dinakdugar dinakdugar force-pushed the 18.0-sales-gelato-dikd branch from 0c5ce5e to cccdc8e Compare February 19, 2025 18:15
@dinakdugar dinakdugar requested a review from jero-odoo February 19, 2025 18:35
@dinakdugar
Copy link
Contributor Author

Hi @jero-odoo this PR is ready for peer review, when you have time. Thank you!

Copy link
Contributor

@jero-odoo jero-odoo left a comment

Choose a reason for hiding this comment

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

Hey @dikd-odoo, Looks great to me! Just one suggestion, but other than that this is ready to go to final. Thanks!

Apply suggestions from JERO review

Co-authored-by: Jess Rogers <104637850+jero-odoo@users.noreply.github.com>

edits
@dinakdugar dinakdugar force-pushed the 18.0-sales-gelato-dikd branch from aa1b76c to 5fbbf4e Compare February 19, 2025 19:59
@dinakdugar
Copy link
Contributor Author

Thank you @jero-odoo for the super quick turnaround and helpful review!

Hi @Felicious this is ready for final review, when you have time. Thank you!

@dinakdugar dinakdugar requested a review from Felicious February 19, 2025 20:41
@StraubCreative StraubCreative requested review from a team and removed request for Felicious February 19, 2025 20:42
Copy link
Contributor

@StraubCreative StraubCreative left a comment

Choose a reason for hiding this comment

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

LGTM 👍
..
@robodoo r+

robodoo pushed a commit that referenced this pull request Feb 20, 2025
Apply suggestions from JERO review

edits

closes #12114

Signed-off-by: Zachary Straub (zst) <zst@odoo.com>
Co-authored-by: Jess Rogers <104637850+jero-odoo@users.noreply.github.com>
robodoo pushed a commit that referenced this pull request Feb 20, 2025
Apply suggestions from JERO review

edits

closes #12114

Signed-off-by: Zachary Straub (zst) <zst@odoo.com>
Co-authored-by: Jess Rogers <104637850+jero-odoo@users.noreply.github.com>
@robodoo robodoo closed this in e32f4ec Feb 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants