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

[WIP] Budget Stats generation & display #1712

Closed
wants to merge 21 commits into from
Closed

Conversation

bertocq
Copy link
Collaborator

@bertocq bertocq commented Jun 28, 2017

Where

  • Related Issues:

What

Madrid's fork got some nice Budget Stats view, on deprecated Spending Proposal model. We want them for the new Budget model and usage.

Getting current specific code from https://github.com/AyuntamientoMadrid/consul/blob/master/app/controllers/spending_proposals_controller.rb#L71 and specific view. And moving on from Geozones usage on to Headings usage for population/district specifics.

How

  • Adding a missing ballot_lines_count counter_cache to Budget::Ballot::Line relationship with Budget::Ballot for balloters stats

  • Adding votes and ballots to Budget::Investments on dev_seeds so local development can try this things. Maybe its not done quite nice or good... need to invest time in dev_seeds to completly get all scenarios done right.

  • Adding a Budget Stats controller, route and ability for anyone to see stats when Budget is either on ballots reviewing or finished phases.

  • Adding a Budget::Stats class and its view (although I need some help with this, since styles are not my thing :D, also many stat names need to be fixed and maybe some style class names as well) @decabeza you MVP!!

  • Adding link to results page on tabs under header

Screenshots

localhost-3000-budgets-7-stats

Test

Incoming from Madrid.

Deployment

As usual

Warnings

Still some bits to fix on the Results side... new PR will follow

@voodoorai2000
Copy link
Member

👏👏👏

@bertocq bertocq mentioned this pull request Jul 10, 2017
4 tasks
@bertocq bertocq force-pushed the feature/budget_stats branch 4 times, most recently from 2003875 to c4312f0 Compare July 11, 2017 17:18
@bertocq bertocq changed the title [WIP] Budget Stats generation & display Budget Stats generation & display Jul 11, 2017
@bertocq bertocq changed the title Budget Stats generation & display [WIP] Budget Stats generation & display Jul 11, 2017
@voodoorai2000 voodoorai2000 mentioned this pull request Aug 5, 2017
@voodoorai2000 voodoorai2000 added this to Next in Roadmap Aug 5, 2017
@voodoorai2000 voodoorai2000 moved this from Next to Doing in Roadmap Sep 6, 2017
@voodoorai2000 voodoorai2000 moved this from Doing to Next in Roadmap Sep 13, 2017
@voodoorai2000 voodoorai2000 removed this from Next in Roadmap Sep 29, 2017
@bertocq bertocq added this to the v0.11 milestone Nov 21, 2017
@bertocq bertocq modified the milestones: v0.11, v0.12 Dec 7, 2017
@bertocq bertocq removed this from the v0.13 milestone Jan 4, 2018
@bertocq
Copy link
Collaborator Author

bertocq commented Apr 6, 2018

We'll be taking a second look at stats in a global level soon, this will be improved then. Frozen for now..

@javierm
Copy link
Member

javierm commented May 29, 2019

Closed in favor of #3438 and #3512.

@javierm javierm closed this May 29, 2019
@javierm javierm deleted the feature/budget_stats branch October 12, 2019 02:33
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.

None yet

4 participants