Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(editor): Redirect Settings to the proper sub page depending on the instance type (cloud or not) #12053

Merged
merged 2 commits into from
Dec 5, 2024

Conversation

cstuncsik
Copy link
Contributor

@cstuncsik cstuncsik commented Dec 4, 2024

Summary

Settings redirection needs to take instance type into account (cloud or not)

Related Linear tickets, Github issues, and Community forum posts

PAY-2334

Review / Merge checklist

  • PR title and summary are descriptive.
  • Tests included.
  • PR Labeled with release/backport

Copy link

codecov bot commented Dec 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

@cstuncsik cstuncsik requested a review from r00gm December 4, 2024 18:29
@n8n-assistant n8n-assistant bot added n8n team Authored by the n8n team ui Enhancement in /editor-ui or /design-system labels Dec 4, 2024
Copy link

cypress bot commented Dec 5, 2024

n8n    Run #8218

Run Properties:  status check passed Passed #8218  •  git commit d22a206278: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 cstuncsik 🗃️ e2e/*
Project n8n
Branch Review pay-2334-cannot-access-settings-anymore
Run status status check passed Passed #8218
Run duration 04m 27s
Commit git commit d22a206278: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 cstuncsik 🗃️ e2e/*
Committer Csaba Tuncsik
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 1
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 480
View all changes introduced in this branch ↗︎

Copy link
Contributor

github-actions bot commented Dec 5, 2024

✅ All Cypress E2E specs passed

@cstuncsik cstuncsik changed the title fix(editor): Open Settings with Personal settings subpage because Usage and plan is unavailable on cloud fix(editor): Redirect Settings to the proper sub page depending on the instance type (cloud or not) Dec 5, 2024
@cstuncsik cstuncsik merged commit a16d006 into master Dec 5, 2024
39 checks passed
@cstuncsik cstuncsik deleted the pay-2334-cannot-access-settings-anymore branch December 5, 2024 07:47
ivov pushed a commit that referenced this pull request Dec 6, 2024
@github-actions github-actions bot mentioned this pull request Dec 6, 2024
@janober
Copy link
Member

janober commented Dec 6, 2024

Got released with n8n@1.71.1

@github-actions github-actions bot mentioned this pull request Dec 11, 2024
# for free to join this conversation on GitHub. Already have an account? # 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