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

cherry-pick(#6943): fix(#6942): Toggling FlexibleLayout toolbar options reflects immediately in the view #6945

Merged
merged 1 commit into from
Aug 16, 2023

Conversation

ozyx
Copy link
Contributor

@ozyx ozyx commented Aug 16, 2023

  • fix: restore reactivity of config settings
  • move initialization steps to created() hook

  • remove unnecessary :key binds

  • fix comments

  • refactor: clean up

  • refactor: clean up

  • refactor: lint:fix

  • test(e2e): add regression test and cleanup suite

  • refactor: consistency is key!

  • test(fix): fix unit tests, further cleanup

Issue #6942

…ns reflects immediately in the view

* fix: restore reactivity of config settings

- move initialization steps to `created()` hook

- remove unnecessary `:key` binds

- fix comments

* refactor: clean up

* refactor: clean up

* refactor: lint:fix

* test(e2e): add regression test and cleanup suite

* refactor: consistency is key!

* test(fix): fix unit tests, further cleanup
@ozyx ozyx added this to the Target:3.0.1 milestone Aug 16, 2023
@ozyx ozyx requested a review from unlikelyzero August 16, 2023 18:20
@deploysentinel
Copy link

deploysentinel bot commented Aug 16, 2023

Current Playwright Test Results Summary

✅ 135 Passing - ⚠️ 3 Flaky

Run may still be in progress, this comment will be updated as current testing workflow or job completes...

(Last updated on 08/16/2023 07:21:05pm UTC)

Run Details

Running Job e2e-stable on CircleCI

Commit: 3b9fb1a

Started: 08/16/2023 07:13:29pm UTC

⚠️ Flakes

📄   functional/plugins/timeConductor/timeConductor.e2e.spec.js • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Time conductor input fields real-time mode preserve offsets and url params when switching between fixed and real-time mode
Retry 1Initial Attempt
0.78% (1) 1 / 129 run
failed over last 7 days
1.55% (2) 2 / 129 runs
flaked over last 7 days

📄   functional/plugins/plot/logPlot.e2e.spec.js • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Log plot tests Log Plot ticks are functionally correct in regular and log mode and after refresh
Retry 1Initial Attempt
2.16% (3) 3 / 139 runs
failed over last 7 days
40.29% (56) 56 / 139 runs
flaked over last 7 days

📄   functional/plugins/conditionSet/conditionSet.e2e.spec.js • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Basic Condition Set Use ConditionSet should output blank instead of the default value
Retry 1Initial Attempt
14.16% (16) 16 / 113 runs
failed over last 7 days
30.09% (34) 34 / 113 runs
flaked over last 7 days

View Detailed Build Results


Current Playwright Test Results Summary

✅ 14 Passing

Run may still be in progress, this comment will be updated as current testing workflow or job completes...

(Last updated on 08/16/2023 07:21:05pm UTC)

Run Details

Running Workflow e2e-couchdb on Github Actions

Commit: 3b9fb1a

Started: 08/16/2023 06:24:10pm UTC

View Detailed Build Results


@unlikelyzero unlikelyzero merged commit 5619994 into release/3.0.1 Aug 16, 2023
@unlikelyzero unlikelyzero deleted the cherry-pick-6943 branch August 16, 2023 20:51
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