[gold] Force reload page after add/edit
Change-Id: I18d8b59f00bd318d258d5e757d34e377996f4759
Bug: skia:9525
Reviewed-on: https://skia-review.googlesource.com/c/buildbot/+/263944
Reviewed-by: Leandro Lovisolo <lovisolo@google.com>
diff --git a/golden/frontend/res/imp/ignores-page-sk.html b/golden/frontend/res/imp/ignores-page-sk.html
index e778eea..34d4f8e 100644
--- a/golden/frontend/res/imp/ignores-page-sk.html
+++ b/golden/frontend/res/imp/ignores-page-sk.html
@@ -268,9 +268,9 @@
_sendRule: function(url) {
this._currRule.filter = this.$.queryInput.currentquery;
- sk.post(url, JSON.stringify(this._currRule)).then(JSON.parse).then(function(json) {
- this._displayRules(json);
+ sk.post(url, JSON.stringify(this._currRule)).then(function() {
this._closeDialog();
+ this.pageSelected();
}.bind(this)).catch(function(e) {
this._closeDialog();
sk.errorMessage(e);
diff --git a/golden/go/web/web.go b/golden/go/web/web.go
index a7f9689..4ce2593 100644
--- a/golden/go/web/web.go
+++ b/golden/go/web/web.go
@@ -808,8 +808,10 @@
return
}
- // If update worked just list the current ignores and return them.
- wh.IgnoresHandler(w, r)
+ sklog.Infof("Successfully updated ignore with id %s", id)
+ if _, err := w.Write([]byte(`{"updated":"true"}`)); err != nil {
+ sklog.Warningf("error responding success to update: %s", err)
+ }
}
// IgnoresDeleteHandler deletes an existing ignores rule.
@@ -870,7 +872,10 @@
return
}
- wh.IgnoresHandler(w, r)
+ sklog.Infof("Successfully added ignore from %s", user)
+ if _, err := w.Write([]byte(`{"added":"true"}`)); err != nil {
+ sklog.Warningf("error responding success to added: %s", err)
+ }
}
// TriageHandler handles a request to change the triage status of one or more