[IMP] bi_sql_editor : conserve cron settings, when setting materialized view to draft ; Set correct default values for cron and enable it

[legalsylvain]

Use case :

• create materialized views, create models and configure the cron
• reset to draft to change request. (adding / removing fields)
• recreate models

-> the cron settings are lost and has to be set again.

With that patch :

• cron are kept if materialized views are set to draft
• default run values are now daily at midnight, beginning tomorrow (before was monthly with the datetime of the validation of the view that was making the cron to be refreshed immediately)
• cron is now active by default. (and disabled when view is reset to draft)

CC @quentinDupont

apps/deck/#/board/144/card/1611

[etobella]

Makes sense

[tfossoul]

@legalsylvain Hi, I would like to have this fix in v13. What is the right approach to do it ? Can I just cherry-pick your commit (+ fix eventual migration bug) into a new v13 branch and do a pull request ?

[legalsylvain]

/ocabot merge patch

@legalsylvain Hi, I would like to have this fix in v13. What is the right approach to do it ? Can I just cherry-pick your commit (+ fix eventual migration bug) into a new v13 branch and do a pull request ?

yes it should work.

[OCA-git-bot]

Sorry @legalsylvain you are not allowed to merge.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

[legalsylvain]

damn'it, not defined as maintainer of this module.

[legalsylvain]

now it is.

/ocabot merge patch

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 12.0-ocabot-merge-pr-519-by-legalsylvain-bump-patch, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 5f022a5. Thanks a lot for contributing to OCA. ❤️