Merge ~ya-bo-ng/maas:fix-view-full-history into maas:master

Proposed by Anthony Dillon
Status: Merged
Approved by: Andres Rodriguez
Approved revision: e390cd576dbaca7d410fe5ff672d2465cd3cf340
Merge reported by: MAAS Lander
Merged at revision: not available
Proposed branch: ~ya-bo-ng/maas:fix-view-full-history
Merge into: maas:master
Diff against target: 21 lines (+1/-2)
1 file modified
src/maasserver/static/partials/node-events.html (+1/-2)
Reviewer Review Type Date Requested Status
Blake Rouse (community) Approve
Karl Williams (community) Approve
MAAS Lander Approve
Andres Rodriguez (community) Needs Information
Review via email: mp+337969@code.launchpad.net

Commit message

LP: #1749863 - Change the event object iterated on for the view full history of events page

Description of the change

The access to the events object had been changed during the Vanilla transition. I have reverted it back to how it was before and is working without errors in the console now.

To post a comment you must log in.
Revision history for this message
Anthony Dillon (ya-bo-ng) wrote :
Revision history for this message
Andres Rodriguez (andreserl) wrote :

@Ant, please set a commit message here in LP with the following format

LP: #xxxxx - <short description>

<long description of needed >

Revision history for this message
Andres Rodriguez (andreserl) wrote :

Quick question inline:

review: Needs Information
Revision history for this message
MAAS Lander (maas-lander) wrote :

UNIT TESTS
-b fix-view-full-history lp:~ya-bo-ng/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: SUCCESS
COMMIT: e390cd576dbaca7d410fe5ff672d2465cd3cf340

review: Approve
Revision history for this message
Anthony Dillon (ya-bo-ng) wrote :

Thanks for the feedback Andres. Sorry I was rushing between meetings. I will take more time to file MP properly next time.

Revision history for this message
Karl Williams (deadlight) wrote :

Looks good and confirmed working.

review: Approve
Revision history for this message
Blake Rouse (blake-rouse) wrote :

Looks good.

review: Approve
Revision history for this message
MAAS Lander (maas-lander) wrote :

UNABLE TO START LANDING

STATUS: MISSING COMMIT MESSAGE

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/src/maasserver/static/partials/node-events.html b/src/maasserver/static/partials/node-events.html
2index be1c99f..59e3e91 100644
3--- a/src/maasserver/static/partials/node-events.html
4+++ b/src/maasserver/static/partials/node-events.html
5@@ -39,7 +39,6 @@
6 class="p-search-box__input" data-ng-model="search" />
7 <button type="submit" class="p-search-box__button" alt="search"><i class="p-icon--search"></i></button>
8 </div>
9-
10 <table id="events-listing" window-width>
11 <thead data-ng-if="windowWidth >= 769" offset="88" media-query="min-width: 768px">
12 <tr>
13@@ -48,7 +47,7 @@
14 </tr>
15 </thead>
16 <tbody vs-repeat vs-scroll-parent="window">
17- <tr data-ng-repeat="event in node.events | filter:search | orderByDate:'created':'id' track by event.id">
18+ <tr data-ng-repeat="event in events | filter:search | orderByDate:'created':'id' track by event.id">
19 <td class="col-9">
20 <span class="p-icon--{$ event.type.level $}"></span>
21 {$ getEventText(event) $}

Subscribers

People subscribed via source and target branches