Skip to content

Split dashboard into widgets. Add build errors widget.#131

Merged
corpsee merged 1 commit into
php-censor:masterfrom
ss-gxp:dashboard-widgets
Nov 7, 2017
Merged

Split dashboard into widgets. Add build errors widget.#131
corpsee merged 1 commit into
php-censor:masterfrom
ss-gxp:dashboard-widgets

Conversation

@ss-gxp

@ss-gxp ss-gxp commented Nov 7, 2017

Copy link
Copy Markdown
Member

@corpsee corpsee added the feature label Nov 7, 2017
@corpsee

corpsee commented Nov 7, 2017

Copy link
Copy Markdown
Member

It is very impression feature 😍! Thank you @ss-gxp! I will try to review and test this request today+tomorrow.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess it would be 'last_builds'?

@ss-gxp ss-gxp Nov 7, 2017

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed, thanks

@ss-gxp ss-gxp force-pushed the dashboard-widgets branch from 95222dc to e29e6c7 Compare November 7, 2017 12:08

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to filter archived projects somewhere here. Like in WidgetAllProjectsController.php (Line 107).

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Filter added (at contoller level, not store level, to avoid join)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, controller level is ok.

@ss-gxp ss-gxp force-pushed the dashboard-widgets branch from e29e6c7 to 11f58d7 Compare November 7, 2017 15:14
@corpsee corpsee merged commit 08e3b56 into php-censor:master Nov 7, 2017
@ss-gxp ss-gxp deleted the dashboard-widgets branch November 7, 2017 15:42
@vinpel

vinpel commented Nov 8, 2017

Copy link
Copy Markdown
Member

Dashboard visualy empty on my isntallation
and got 2 javascript error (chrome) :

Uncaught TypeError: Cannot set property 'allProjects' of undefined
    at all_projects.js:1
(anonymous) @ all_projects.js:1
last_builds.js:1 Uncaught TypeError: Cannot set property 'lastBuilds' of undefined
    at last_builds.js:1
(anonymous) @ last_builds.js:1

@corpsee

corpsee commented Nov 8, 2017

Copy link
Copy Markdown
Member

Thank you @vinpel. @ss-gxp Can you try to see what is wrong?

@ss-gxp

ss-gxp commented Nov 8, 2017

Copy link
Copy Markdown
Member Author

I think this browser cache. Try Ctrl-F5

@vinpel

vinpel commented Nov 8, 2017

Copy link
Copy Markdown
Member

CTRL+F5 solve the problem !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants