-
Notifications
You must be signed in to change notification settings - Fork 20
Agency view: finish integrating bar charts #152
Comments
@harrisj depending on our priorities this next sprint, can we talk about the merits of implementing a better solution for aggregating the sub offenses on the backend versus the front end? |
I'd like to talk about this once we know the other things we're looking at for this sprint. At this point in the project, I don't know if we should be doing aggregation client side. The reason is because it just papers over a usability problem with the API that other folks are going to run into. Better to just fix the API IMO |
@jeremiak, i think @harrisj agrees and is planning to implement a fix on his end. @harrisj what do you think? if you think its doable, we can create a sub tasks for this issue and start on that right away. the goal is to resolve all dependencies for the front-end team so they can finish building out the agency view this sprint. |
@jeremiak is this a design task to review. I'm not sure I can comment on this. I'll move it into to review but feel free to close if it is complete or let me know what I should comment on. |
This task includes the ability to download displayed information as a CSV
The text was updated successfully, but these errors were encountered: