-
Notifications
You must be signed in to change notification settings - Fork 32
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
Contracts dashboards: assignees ranking #3075
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3075 +/- ##
=======================================
Coverage 77.42% 77.42%
=======================================
Files 788 788
Lines 21294 21294
=======================================
Hits 16486 16486
Misses 4808 4808 Continue to review full report at Codecov.
|
19941f0
to
acb5028
Compare
Is this another topic branch? I added a |
…reusable when filtering
Added new generic modules for the charts
They were broken after changing 1) tenders/contracts default order and 2) only making calculations only with formalized contracts.
keep-alive was being rendered twice and cache was happening twice.
…ntractsIndex and TendersIndex Also, aligning quantity columns to the right.
8edd2f1
to
71e0b01
Compare
@Crashillo this can now be reviewed 👍 |
app/javascript/gobierto_dashboards/modules/contracts_controller.js
Outdated
Show resolved
Hide resolved
app/javascript/gobierto_dashboards/modules/contracts_controller.js
Outdated
Show resolved
Hide resolved
app/javascript/gobierto_dashboards/modules/contracts_controller.js
Outdated
Show resolved
Hide resolved
app/javascript/gobierto_dashboards/webapp/containers/contract/ContractsShow.vue
Outdated
Show resolved
Hide resolved
app/javascript/gobierto_dashboards/webapp/containers/summary/Summary.vue
Show resolved
Hide resolved
app/javascript/gobierto_dashboards/webapp/containers/summary/Summary.vue
Outdated
Show resolved
Hide resolved
Co-authored-by: Hugoren Martinako <[email protected]>
…ierto into 3046-assignees-ranking
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You may include the suggestion in the next (derived) PR
app/javascript/gobierto_dashboards/webapp/containers/contract/ContractsShow.vue
Outdated
Show resolved
Hide resolved
Co-authored-by: Hugoren Martinako <[email protected]>
Closes #3046
✌️ What does this PR do?
It mainly adds the assignees ranking to the Dashboards contracts summary page. It should refresh along with the bar charts filters.
Along with that, it also includes the following fixes and features:
🔍 How should this be manually tested?
It can be tested from the main summary page. It's been deployed to staing https://burjassot.gobify.net/dashboards/contratos/resumen
👀 Screenshots
.env.example
?config/application.yml
?config/secrets.yml
?(Changes in these files might need to update the role in Ansible)
📖 Does this PR require updating the documentation?