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

Add basic chart widget to API user dashboard #120

Closed
1 of 2 tasks
bajiat opened this issue Jun 12, 2015 · 2 comments
Closed
1 of 2 tasks

Add basic chart widget to API user dashboard #120

bajiat opened this issue Jun 12, 2015 · 2 comments
Assignees
Milestone

Comments

@bajiat
Copy link
Contributor

bajiat commented Jun 12, 2015

Using chart widget from task #78, add placeholder chart to API user dashboard.

Definition of done

  • Insert chart template into API user dashboard template.
  • Add any additional markup necessary.
@55 55 self-assigned this Jun 14, 2015
@brylie brylie assigned frenchbread and unassigned 55 Jun 15, 2015
@brylie brylie added this to the Sprint 5 milestone Jun 15, 2015
@frenchbread
Copy link
Contributor

Currently waiting for @elnzv to implement a sidebar for API user dashboard. So far I moved chart template to dashboard and removed routes and templates for posts. Here is the branch.

@frenchbread
Copy link
Contributor

Since the data from elastic search is accessible, I am now trying to pass the data from the server-side (elasticSearch.js file) to template on client-side through the router (main.js). For that approach, @elnzv suggested to use reacrive-var package. Its functionality is very similar to Session. But the problem is that when I am assigning a variable inside ES.search method, it is not possible to read it outside the function, meaning that it comes as undefined. ElasticSearch package's documentation does not provide any additional info at this point.

@brylie, Any suggestions or possible solutions? This is actually javascript issue, nothing to do with meteor here. Global variables did not work as well as Sessions.
Here is working branch and last commit.

brylie added a commit that referenced this issue Dec 20, 2016
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

No branches or pull requests

4 participants