Skip to content

[FIX] bi_sql_editor : handle correctly translation for action name - #455

Merged
OCA-git-bot merged 1 commit into
OCA:12.0from
legalsylvain:12.0-FIX-bi_sql_editor-action-name-translated
Nov 26, 2021
Merged

OCA-git-bot merged 1 commit into
OCA:12.0from
legalsylvain:12.0-FIX-bi_sql_editor-action-name-translated

Conversation

@legalsylvain

@legalsylvain legalsylvain commented Nov 12, 2020

Copy link
Copy Markdown
Contributor

Trivial patch.

without this patch, when the cron is executing the refresh of the materialized view, the name of the action is not correctly updated.

@alan196 alan196 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Technicall Review

@legalsylvain

Copy link
Copy Markdown
Contributor Author

@quentinDupont : could you take a look on this one ? it is on production since November.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@dreispt

dreispt commented Nov 26, 2021

Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@OCA-git-bot
OCA-git-bot merged commit 013bf73 into OCA:12.0 Nov 26, 2021
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants