Skip to content

WIP: Create Countdown React vis type#11984

Closed
cjcenizal wants to merge 73 commits intoelastic:feature/visualizefrom
cjcenizal:experiment/react-visualization-api
Closed

WIP: Create Countdown React vis type#11984
cjcenizal wants to merge 73 commits intoelastic:feature/visualizefrom
cjcenizal:experiment/react-visualization-api

Conversation

@cjcenizal
Copy link
Copy Markdown
Contributor

No description provided.

@cjcenizal cjcenizal added the Feature:Visualizations Generic visualization features (in case no more specific feature label is available) label May 24, 2017
@cjcenizal
Copy link
Copy Markdown
Contributor Author

@ppisljar @thomasneirynck Any idea why I'm getting vis and visData as undefined in the render method of react_vis_type.js?

@cjcenizal cjcenizal force-pushed the experiment/react-visualization-api branch from 43a9d6e to 6774d5d Compare May 24, 2017 14:59
@ppisljar
Copy link
Copy Markdown
Contributor

is this recently rebased ? (like .... 3 hours ago)

@cjcenizal
Copy link
Copy Markdown
Contributor Author

@ppisljar Yep, rebased 20 mins ago

@ppisljar
Copy link
Copy Markdown
Contributor

visData is undefined because your response_handler resolves() with no parameters ... you should pass a data parameter to it (at least resolve(true) then your visData will be true and will indicate that your visualization is data dependent.

vis was null due to a bug in react_vis_types which is now fixed (pls rebase)

@ppisljar ppisljar force-pushed the feature/visualize branch 9 times, most recently from 0e3d49f to 8f9f1db Compare June 9, 2017 08:54
@ppisljar ppisljar force-pushed the feature/visualize branch 4 times, most recently from 63e1421 to abf0a6d Compare June 29, 2017 13:51
@ppisljar ppisljar force-pushed the feature/visualize branch from 20bf21c to 033806a Compare July 7, 2017 14:47
@cjcenizal cjcenizal closed this Jul 10, 2017
@cjcenizal cjcenizal deleted the experiment/react-visualization-api branch July 10, 2017 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Visualizations Generic visualization features (in case no more specific feature label is available)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants