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
[14.0] [MIG] web_no_bubble migration to 14.0 #1845
Conversation
Includes some manual fixes to silent ESLint warnings.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code review looks good to me.
Hi @ellbristow it would be great you can ask someone to add review to get merge thi PR soon. |
@ALL can somone please review the PR so that we can merge it ASAP |
@OCA/web-maintainers can somone review the code and merge the PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR has the |
It would be great if maintainers could merge the PR, Hence it is approved. |
It would be great if maintainers could merge the PR. |
It would be great if maintainers could merge the PR. |
@OCA/web-maintainers can you please merge this PR? |
It would be great if the mainainers merge the PR. |
/ocabot merge nobump |
On my way to merge this fine PR! |
Congratulations, your PR was merged at 0d0355d. Thanks a lot for contributing to OCA. ❤️ |
Pushed wrong branch in #1839
Second attempt
Added: Apply no_bubble template to frontend web editor as well as backend