Skip to content

Commit 56192b9

Browse files
authored
Merge branch 'master' into feature/new-plus-button
2 parents 0c477c3 + 483d4e4 commit 56192b9

File tree

2 files changed

+20
-6
lines changed

2 files changed

+20
-6
lines changed

.github/mergify.yml

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,20 +24,30 @@ queue_rules:
2424

2525
# NOTE: in case of flaky tests you above checks will fail
2626
# the PR will be removed from the queue.
27-
# This retries the checks, as soon as they become green,
28-
# the PR will be pusehd to the queu again
27+
# Once the PR is ready to be merged the flaky tests will
28+
# be retried automatically.
29+
# As soon as the CI is green, the PR will be pushed to the queue
2930
pull_request_rules:
30-
- name: retry CI on fails (used for flaky tests)
31+
- name: retry flaky tests (when PR is approved)
3132
conditions:
3233
- label=🤖-automerge
3334
- label!=🤖-do-not-merge
3435
- base=master
36+
37+
- -draft # PR is not in draft state
38+
- -conflict # No merge conflicts
39+
- "#approved-reviews-by>=2" # Requires 2 approving reviews
40+
- "#changes-requested-reviews-by=0" # No changes requested
41+
- "#review-threads-unresolved=0" # All review threads resolved
42+
3543
- or:
3644
- check-failure=unit-tests
3745
- check-failure=integration-tests
3846
- check-failure=system-tests
3947
actions:
48+
comment:
49+
message: "Restarted CI due to failing tests"
4050
github_actions:
4151
workflow:
4252
dispatch:
43-
- workflow: .github/workflows/ci-testing-deploy.yml
53+
- workflow: ci-testing-deploy.yml

services/static-webserver/client/source/class/osparc/auth/LoginPageSplit.js

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,9 @@ qx.Class.define("osparc.auth.LoginPageSplit", {
2525
const layout = new qx.ui.layout.HBox();
2626
this._setLayout(layout);
2727

28+
const loginPage = this.__loginPage = new osparc.auth.LoginWithDecorators();
29+
loginPage.addListener("done", e => this.fireDataEvent("done", e.getData()));
30+
2831
this.__rebuildLayout();
2932

3033
setTimeout(() => this.__resized(), 100);
@@ -51,6 +54,8 @@ qx.Class.define("osparc.auth.LoginPageSplit", {
5154
},
5255

5356
members: {
57+
__loginPage: null,
58+
5459
_getBackgroundImage: function() {
5560
throw new Error("Abstract method called!");
5661
},
@@ -67,8 +72,7 @@ qx.Class.define("osparc.auth.LoginPageSplit", {
6772
__rebuildLayout: function() {
6873
this._removeAll();
6974

70-
const loginPage = new osparc.auth.LoginWithDecorators();
71-
loginPage.addListener("done", e => this.fireDataEvent("done", e.getData()));
75+
const loginPage = this.__loginPage;
7276
const container = new qx.ui.container.Scroll();
7377
container.add(loginPage);
7478
const spacers = [

0 commit comments

Comments
 (0)