n8n icon indicating copy to clipboard operation
n8n copied to clipboard

feat(editor): Remove templates from n8n app

Open MiloradFilipovic opened this issue 1 year ago • 3 comments

Summary

Removing templates repository from n8n app and pointing users to website instead. This PR should remove most of the template components except for the ones related to template setup and ones used to suggest templates.

Related tickets and issues

Closes ADO-1789

Review / Merge checklist

  • [ ] PR title and summary are descriptive. Remember, the title automatically goes into the changelog. Use (no-changelog) otherwise. (conventions)
  • [ ] Docs updated or follow-up ticket created.
  • [ ] Tests included.

    A bug is not considered fixed, unless a test is added to prevent it from happening again. A feature is not complete without tests.

MiloradFilipovic avatar Feb 07 '24 10:02 MiloradFilipovic

:warning: Some Cypress E2E specs are failing, please fix them before merging

github-actions[bot] avatar Feb 07 '24 11:02 github-actions[bot]

1 flaky test on run #4036 ↗︎

0 334 5 0 Flakiness 1

Details:

🌳 🖥️ browsers:node18.12.0-chrome107 🤖 MiloradFilipovic 🗃️ e2e/*
Project: n8n Commit: 0d88551faf
Status: Passed Duration: 03:31 💡
Started: Feb 7, 2024 11:46 AM Ended: Feb 7, 2024 11:50 AM
Flakiness  cypress/e2e/5-ndv.cy.ts • 1 flaky test

View Output Video

Test Artifacts
NDV > should not retrieve remote options when required params throw errors Test Replay Screenshots Video

Review all test suite changes for PR #8565 ↗︎

cypress[bot] avatar Feb 07 '24 11:02 cypress[bot]

:white_check_mark: All Cypress E2E specs passed

github-actions[bot] avatar Feb 07 '24 11:02 github-actions[bot]

Closing in favor of #8591

MiloradFilipovic avatar Feb 08 '24 13:02 MiloradFilipovic