Skip to content
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

Syncing from upstream OCA/reporting-engine (13.0) #716

Merged
merged 3 commits into from
Sep 3, 2024
Merged

Conversation

bt-admin
Copy link

@bt-admin bt-admin commented Sep 3, 2024

bt_gitbot

GuillemCForgeFlow and others added 3 commits September 2, 2024 12:56
…etting the SQL View

Based on the agreed solution: #903 (review). It is best to archive the Scheduled Action every time you set back to the draft state. Then, when creating the SQL view, if it is a materialized view, unarchive the Scheduled Action if already exists, otherwise create it from scratch.
Then, if the SQL View is deleted, also make sure to remove the Scheduled Action if there is one assigned as it has not been done in the `button_set_draft` method
Signed-off-by legalsylvain
@bt-admin bt-admin added the 13.0 label Sep 3, 2024
@bt-admin bt-admin merged commit 5efa3a6 into brain-tec:13.0 Sep 3, 2024
4 checks passed
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.

3 participants