-
Notifications
You must be signed in to change notification settings - Fork 238
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
As a develop I want the data in the chart to change in response to other actions on the page #13
Comments
In discussions w/ @ajturner we arrived at the idea that the "compiled" template is just the template w/ the mappings attached. The model shown above also implies that a "cedar chart" is an object w/ state & behavior, thus we should be doing something like:
|
So the query may include extent and where filters based on application functionality and the chart requires a group by attribute that populates the query within cedar. The benefit is that the app developer focuses on functionality and cedar handles aggregations and statistics specific to the chart viz. |
Agreed. those properties would likely be getter/setter functions so you don't need to specify anything but if you want to see current settings you can get at them. This is more accurate...
|
First steps of this are in #17 |
The plumbing for this is inplace - need to build out an example showing the query being updated based on other ui inputs (map extent or some form elements) |
@dbouwman @ajturner this is done right? Even with example of query being set by map extent: http://esridc.github.io/cedar/examples/filter-chart-by-map-extent.html Closing. |
Generalized from #12
Pseudo code
Cedar Impact
So, the first part of this would be to hold the mappings in the compiled chart vs actually replacing them in the template.
The second part would be to expose the query as part of the chart, and the ability to call .update(queryParams)
Chart Template Inputs
The inputs define the required
Compiled Chart Inputs have values stored
The text was updated successfully, but these errors were encountered: