Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[IMP] studio: updating automation rules and webhooks #13059

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

[IMP] studio: updating automation rules and webhooks #13059

wants to merge 1 commit into from

Conversation

mcsi-odoo
Copy link
Contributor

Task-4672373

@robodoo
Copy link
Collaborator

robodoo commented Apr 25, 2025

Pull request status dashboard

@C3POdoo C3POdoo requested a review from a team April 25, 2025 08:13
@mcsi-odoo mcsi-odoo requested review from cth-odoo and removed request for a team April 25, 2025 08:26
@mcsi-odoo mcsi-odoo force-pushed the 18.0-studio-automation-rules-webhooks-mcsi branch from ee6588c to 681647b Compare May 27, 2025 11:10
@mcsi-odoo mcsi-odoo requested review from a team and removed request for cth-odoo May 27, 2025 12:35
@auva-odoo auva-odoo requested review from auva-odoo and removed request for a team May 28, 2025 07:07
auva-odoo
auva-odoo requested changes May 28, 2025
View reviewed changes
Copy link
Contributor

@auva-odoo auva-odoo left a comment

Choose a reason for hiding this comment

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

Hi @mcsi-odoo, thank you so much for your fantastic work on this! 🙌
I've added a few comments and suggestions, particularly in the Webhooks file where I suggested refining some of the existing content a bit further.
I’ve also included a few questions/discussion points (some are just out of curiosity, and others because I wasn’t fully confident in a solution and would really like your opinion and brain power!)

Thanks again for taking care of this so quickly!

@mcsi-odoo mcsi-odoo force-pushed the 18.0-studio-automation-rules-webhooks-mcsi branch from 681647b to ba0894e Compare June 27, 2025 06:39
@mcsi-odoo mcsi-odoo requested a review from cth-odoo June 27, 2025 07:03
@mcsi-odoo mcsi-odoo force-pushed the 18.0-studio-automation-rules-webhooks-mcsi branch from ba0894e to 27d5b93 Compare June 27, 2025 13:53
@mcsi-odoo mcsi-odoo requested review from auva-odoo and removed request for cth-odoo June 27, 2025 13:58
auva-odoo
auva-odoo requested changes Jul 14, 2025
View reviewed changes
Copy link
Contributor

@auva-odoo auva-odoo left a comment

Choose a reason for hiding this comment

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

Thank you for the updates @mcsi-odoo! And thank you for your patience while I try to catch up! I only have a few comments, mostly minor ones 🙂

mcsi-odoo
mcsi-odoo commented Jul 15, 2025
View reviewed changes
Copy link
Contributor Author

@mcsi-odoo mcsi-odoo left a comment

Choose a reason for hiding this comment

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

Most comments resolved after suggested change implemented.
I replied on some, and will share a new version shortly - main update to note is clarification on the 240 mins vs 2400 mins.

@mcsi-odoo mcsi-odoo force-pushed the 18.0-studio-automation-rules-webhooks-mcsi branch from 27d5b93 to e38abe7 Compare July 15, 2025 07:59
@C3POdoo C3POdoo requested a review from a team July 15, 2025 08:01
@mcsi-odoo mcsi-odoo requested review from auva-odoo and removed request for a team July 15, 2025 09:00
auva-odoo
auva-odoo approved these changes Jul 15, 2025
View reviewed changes
Copy link
Contributor

@auva-odoo auva-odoo left a comment

Choose a reason for hiding this comment

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

LGTM @mcsi-odoo ! Thank you for the changes and clarification 🙂

@robodoo r+

@robodoo robodoo closed this in cf1e633 Jul 15, 2025
This was referenced Jul 15, 2025
Closed
Closed
Open
Open
@fw-bot fw-bot mentioned this pull request Jul 15, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@auva-odoo auva-odoo auva-odoo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mcsi-odoo @robodoo @auva-odoo