Skip to content

fix: update webhook trigger's code example and custom UI on auth change#3135

Merged
forestileao merged 4 commits intosuperplanehq:mainfrom
devroy10:webhook-native-trigger
Feb 25, 2026
Merged

fix: update webhook trigger's code example and custom UI on auth change#3135
forestileao merged 4 commits intosuperplanehq:mainfrom
devroy10:webhook-native-trigger

Conversation

@devroy10
Copy link
Contributor

@devroy10 devroy10 commented Feb 16, 2026

Description

fixes #2995

This makes the webhook trigger settings sidebar render its custom UI using the current form configuration rather than the saved node config. As a result, the auth method, code example, and reset token/signature controls update immediately when the user changes the Authentication dropdown, without requiring a canvas save.

Before

before-dynamic-update.webm

After

after-dynamic-update.webm

@devroy10 devroy10 marked this pull request as ready for review February 16, 2026 17:16
@devroy10 devroy10 changed the title fix: update webhook code example and custom UI on auth change fix: update webhook trigger's code example and custom UI on auth change Feb 16, 2026
@devroy10
Copy link
Contributor Author

devroy10 commented Feb 16, 2026

@AleksandarCole minor UX improvement here.

@devroy10
Copy link
Contributor Author

@shiroyasha could you take a look at this

@AleksandarCole AleksandarCole added the pr:stage-3/3 Ready for full, in-depth, review label Feb 25, 2026
@AleksandarCole
Copy link
Collaborator

Thanks @devroy10!

@forestileao can we check and merge this one? It's going stale.

@devroy10
Copy link
Contributor Author

Thanks @devroy10!

@forestileao can we check and merge this one? It's going stale.

Glad it helps

@forestileao
Copy link
Collaborator

@devroy10 probably needs to fix conflicts

@devroy10
Copy link
Contributor Author

@devroy10 probably needs to fix conflicts

On it

@devroy10
Copy link
Contributor Author

devroy10 commented Feb 25, 2026

@devroy10 probably needs to fix conflicts

@forestileao Conflict resolved, still works as expected

@forestileao forestileao merged commit bf4920e into superplanehq:main Feb 25, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:stage-3/3 Ready for full, in-depth, review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Webhook code example static on auth change

3 participants