OCA
[17.0][IMP]bi_sql_editor: scheduled action periodicity settings
refactor Action Settings page to have all settings, renamed to Settings. Added the settings to be able to customize the periodicity of the Scheduled Action that will refresh the Materialized view.
cc @ForgeFlow
Hi @legalsylvain, some modules you are maintaining are being modified, check this out!
Hi @legalsylvain, I'd like to get your input on this whenever you have some time thank you in advance 🙂
Hi @GuillemCForgeFlow. Thanks for contributing. I don't get the interest of this change. (Except that all cron information are managed in the SQL view.) Could you elaborate ? Thanks !
Hi @GuillemCForgeFlow. Thanks for contributing. I don't get the interest of this change. (Except that all cron information are managed in the SQL view.) Could you elaborate ? Thanks !
Sure, the use case for this is that the periodicity of the refresh was too low. It could be changed at Scheduled Action level, but each time the view gets recreated (because the SQL definition changes) the Scheduled Action gets back to the original parameters.
I thought it would be great to have a general settings tab, in order to fulfill this kind of information if it needs to be changed based on specific needs. We can add more settings' groups on that tab if required in the future.
What do you think?
but each time the view gets recreated (because the SQL definition changes) the Scheduled Action gets back to the original parameters
You're right. That's the point. It should be changed, for exemple, disabling the cron instead of deleting it. What do you think ? It avoid to duplicate fields in both modèls.
I agree, that seems to solve the issue for both of us. I will adjust the changes then to have this in place. Thank you for the input 😄
closing, I will reference the implemented solution once done 👍🏿 @legalsylvain
Implemented agreed changes on https://github.com/OCA/reporting-engine/pull/928