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

fix(editor): More dark-mode fixes (no-changelog) #7624

Merged
merged 2 commits into from
Nov 6, 2023
Merged

Conversation

netroy
Copy link
Member

@netroy netroy commented Nov 6, 2023

This fixes:

  1. The OR divider background when SSO Login is enabled
  2. Scanning of QR code in settings in dark mode

@netroy netroy requested a review from mutdmour November 6, 2023 12:50
mutdmour
mutdmour previously approved these changes Nov 6, 2023
@n8n-assistant n8n-assistant bot added n8n team Authored by the n8n team ui Enhancement in /editor-ui or /design-system labels Nov 6, 2023
Copy link

cypress bot commented Nov 6, 2023

1 flaky test on run #2736 ↗︎

0 270 0 0 Flakiness 1

Details:

🌳 🖥️ browsers:node18.12.0-chrome107 🤖 netroy 🗃️ e2e/*
Project: n8n Commit: 599bcbe581
Status: Passed Duration: 07:49 💡
Started: Nov 6, 2023 1:05 PM Ended: Nov 6, 2023 1:13 PM
Flakiness  cypress/e2e/28-resource-mapper.cy.ts • 1 flaky test

View Output Video

Test Artifacts
Resource Mapper > should correctly delete single field Output Screenshots Video

Review all test suite changes for PR #7624 ↗︎

Copy link
Contributor

github-actions bot commented Nov 6, 2023

✅ All Cypress E2E specs passed

1 similar comment
Copy link
Contributor

github-actions bot commented Nov 6, 2023

✅ All Cypress E2E specs passed

Copy link

codecov bot commented Nov 6, 2023

Codecov Report

Attention: 8 lines in your changes are missing coverage. Please review.

Comparison is base (26361df) 32.08% compared to head (599bcbe) 32.02%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7624      +/-   ##
==========================================
- Coverage   32.08%   32.02%   -0.06%     
==========================================
  Files        3410     3410              
  Lines      208706   208706              
  Branches    22659    22660       +1     
==========================================
- Hits        66971    66848     -123     
- Misses     140570   140693     +123     
  Partials     1165     1165              
Files Coverage Δ
packages/editor-ui/src/components/SSOLogin.vue 89.83% <100.00%> (ø)
packages/editor-ui/src/views/MfaView.vue 0.00% <ø> (ø)
...ackages/editor-ui/src/components/MfaSetupModal.vue 0.00% <0.00%> (ø)

... and 4 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@netroy netroy merged commit 4934462 into master Nov 6, 2023
55 of 56 checks passed
@netroy netroy deleted the dark-mode-fixes branch November 6, 2023 13:32
krynble pushed a commit that referenced this pull request Nov 7, 2023
This fixes:
1. The `OR` divider background when SSO Login is enabled
2. Scanning of QR code in settings in dark mode
@janober
Copy link
Member

janober commented Nov 7, 2023

Got released with n8n@1.15.2

MiloradFilipovic added a commit that referenced this pull request Nov 8, 2023
* master: (24 commits)
  fix(Crypto Node): Fix issue with value not appearing for Sign action (#7619)
  feat(NocoDB Node): Add new data apis and workspace support (#7329)
  fix(Facebook Lead Ads Trigger Node): Fix issue with missing scope for business management  (#7616)
  refactor(core): Stop reporting to Sentry missing-node-on-retry error (no-changelog) (#7648)
  fix(editor): Fix workflow history prune time limit (getting hours instead of days) (#7644)
  fix(core): Comply with custom default for workflow saving settings (#7634)
  feat(core): Initial support for two-way communication over websockets (#7570)
  ci: Conditionally re-enable coverage for frontend packages (no-changelog) (#7641)
  ci(core): Load config schema after process.env has been overwritten (no-changelog) (#7550)
  fix(core): Make password-reset urls valid only for single-use (#7622)
  fix: Error handling on forgot password page (no-changelog) (#7633)
  ci: Improve CI performance (no-changelog) (#7637)
  fix(core): Ensure pruning starts only after migrations have completed (#7626)
  feat(core): Coordinate workflow activation in multiple main scenario in internal API (#7566)
  fix(editor): Fix local storage flags defaulting to undefined string (#7603)
  fix(editor): Reset canvas zoom before workspace reset in node view (#7625)
  fix(editor): More dark-mode fixes (no-changelog) (#7624)
  fix(core): Fix accessor error when running partial execution (#7618)
  fix(editor): Fix issue that frontend breaks with unkown nodes (#7596)
  fix(core): Ensure `init` before checking leader or follower in multi-main scenario (#7621)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
n8n team Authored by the n8n team Released ui Enhancement in /editor-ui or /design-system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants