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

Admin budget investment info #1353

Merged
merged 1 commit into from
Mar 12, 2018
Merged

Admin budget investment info #1353

merged 1 commit into from
Mar 12, 2018

Conversation

decabeza
Copy link

@decabeza decabeza commented Mar 9, 2018

References

Related issue: consuldemocracy#2376

Objectives

If the budget investments have an image or documents now appears a link to budget investment with a #image or #documents anchor on a new tab to allow admin check easily the associated images and docs of the budget investment on evaluation.

Visual Changes

Admin budget investment show before

admin_investment_before

Admin budget investment show after (with image and docs)

screen shot 2018-03-12 at 16 51 47

Admin budget investment show after (without image or docs)

screen shot 2018-03-12 at 16 52 02

Notes

Backport this to CONSUL when merged.

@decabeza decabeza requested a review from voodoorai2000 March 9, 2018 16:00
@decabeza decabeza force-pushed the admin-budget-investments branch from 6ccd810 to fa6fa6e Compare March 12, 2018 16:06
@decabeza decabeza merged commit 5475662 into master Mar 12, 2018
@decabeza decabeza deleted the admin-budget-investments branch March 12, 2018 17:55
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.

2 participants