-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[17.0][MIG] web_widget_bokeh_chart #2846
[17.0][MIG] web_widget_bokeh_chart #2846
Conversation
Avoid "Duplicate explicit target name" error using anonymous hyperlink references Ref.: https://stackoverflow.com/questions/5464627/how-to-have-same-text-in-two-links-with-restructured-text/14067756#14067756
…ontainer element.
…mit dynamic updates
Includes some manual fixes to silent ESLint warnings.
Co-authored-by: Enric Tobella [email protected] Co-authored-by: David Jimenez [email protected]
cc @robyf70 you might want to review this one |
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
Update bokeh from 3.1.1 to 3.4.1
582fc03
to
cef27bc
Compare
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
/ocabot merge nobump |
1 similar comment
/ocabot merge nobump |
Sorry @LoisRForgeFlow 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 |
1 similar comment
Sorry @LoisRForgeFlow 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 |
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.
/ocabot merge nobump
On my way to merge this fine PR! |
/ocabot migration web_widget_bokeh_chart |
Congratulations, your PR was merged at 3e7301b. Thanks a lot for contributing to OCA. ❤️ |
Migration to 17.0
Supersede #2805
Also updates bokeh lib from 3.1.1 to 3.4.1 to include the fix like #2793
@ForgeFlow