Skip to content

Conversation

@michael-radency
Copy link
Contributor

Summary

show pop-up of form trigger only if webhook URL is live

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/NODE-2336/form-trigger-pop-up-shows-when-shouldnt

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

@michael-radency michael-radency added ui Enhancement in /editor-ui or /design-system n8n team Authored by the n8n team labels May 28, 2025
@dana-gill dana-gill requested a review from ShireenMissi May 28, 2025 13:14
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cubic reviewed 3 files and found no issues. Review PR in cubic.dev.

@codecov
Copy link

codecov bot commented May 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

Copy link
Contributor

@ShireenMissi ShireenMissi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested Manually and it works 👌

@shortstacked
Copy link
Contributor

Workflow Test Results 📊 🔴 1 Failed, ⚠️ 2 Warnings, 👍 80 Successful out of 83 total workflows.

Detail: Workflows failing: 243: Workflow contains 2 deleted data. View full workflow run

Tested Ref: 5d3a3f3a1cc9324404346ca33d05462340a48c1e by @ShireenMissi

❌ Failed Tests (1)

Workflow ID Workflow Name Reason
258 Agent:auto-fix:openai Workflow contains 2 deleted data.

⚠️ Warnings (2)

Workflow ID Workflow Name Reason
237 BasicLLMChain:AzureChat Workflow contains new data that previously did not exist.
257 Agent:auto-fix:anthropic Workflow contains new data that previously did not exist.

@github-actions
Copy link
Contributor

✅ All Cypress E2E specs passed

@ShireenMissi ShireenMissi merged commit 041ada1 into master May 29, 2025
21 checks passed
@ShireenMissi ShireenMissi deleted the node-2336-form-trigger-pop-up-shows-when-shouldnt branch May 29, 2025 17:32
@github-actions github-actions bot mentioned this pull request Jun 2, 2025
@janober
Copy link
Member

janober commented Jun 2, 2025

Got released with n8n@1.97.0

Alexandero89 pushed a commit to Alexandero89/n8n that referenced this pull request Jun 4, 2025
Alexandero89 pushed a commit to Alexandero89/n8n that referenced this pull request Jun 4, 2025
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.

5 participants