Skip to content
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

[11.0][MIG] web_widget_bokeh_chart #870

Merged
merged 6 commits into from Feb 19, 2018
Merged

[11.0][MIG] web_widget_bokeh_chart #870

merged 6 commits into from Feb 19, 2018

Conversation

ghost
Copy link

@ghost ghost commented Feb 19, 2018

No description provided.

@pedrobaeza pedrobaeza mentioned this pull request Feb 19, 2018
68 tasks
Copy link
Contributor

@simahawk simahawk left a comment

Choose a reason for hiding this comment

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

LGTM, just a small thing to change in readme example


bokeh_chart = fields.Text(
string='Bokeh Chart',
compute=_compute_bokeh_chart)
Copy link
Contributor

Choose a reason for hiding this comment

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

compute='_compute_bokeh_chart'

@simahawk
Copy link
Contributor

@arkostyuk tests failing because of missing py dependency for bokeh. Add a requirements.txt in the root, like https://github.com/OCA/server-auth/blob/11.0/requirements.txt

@JordiBForgeFlow
Copy link
Member

Please include https://github.com/OCA/web/blob/10.0/.travis.yml#L35 in 11.0. Otherwise Travis complains.

@simahawk
Copy link
Contributor

@jbeficent I'd prefer to state deps clearly into requirements.txt and leave travis conf as clean as possible.

@ghost
Copy link
Author

ghost commented Feb 19, 2018

@simahawk @jbeficent done 😃

@@ -0,0 +1 @@
bokeh==0.12.7
Copy link
Contributor

Choose a reason for hiding this comment

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

@arkostyuk I linked an example to make you spot the comments on top ;)
Add a line a above, like # web_widget_bokeh_chart

@JordiBForgeFlow JordiBForgeFlow merged commit 6756aff into OCA:11.0 Feb 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants