[PR #22284] feat(config-modal): add space to underscore conversion in variable name input of start node #29886

Closed
opened 2026-02-21 20:46:26 -05:00 by yindo · 0 comments
Owner

Original Pull Request: https://github.com/langgenius/dify/pull/22284

State: closed
Merged: Yes


Summary

Add utility function to automatically replace spaces with underscores in variable name input field to ensure compatibility with variable naming conventions. Also includes validation checks for invalid characters.

Screenshots

Before

ab4ee82e90e1499c845a1c80b1345d6

After

https://github.com/user-attachments/assets/386b8366-b7bd-486d-865c-997997605b4b

Checklist

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.
  • I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods
**Original Pull Request:** https://github.com/langgenius/dify/pull/22284 **State:** closed **Merged:** Yes --- ## Summary Add utility function to automatically replace spaces with underscores in variable name input field to ensure compatibility with variable naming conventions. Also includes validation checks for invalid characters. ## Screenshots ### Before <img width="475" height="590" alt="ab4ee82e90e1499c845a1c80b1345d6" src="https://github.com/user-attachments/assets/cdadfd09-1f9f-4b80-8d1f-602124773488" /> ### After https://github.com/user-attachments/assets/386b8366-b7bd-486d-865c-997997605b4b ## Checklist - [ ] This change requires a documentation update, included: [Dify Document](https://github.com/langgenius/dify-docs) - [x] I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!) - [x] I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change. - [x] I've updated the documentation accordingly. - [x] I ran `dev/reformat`(backend) and `cd web && npx lint-staged`(frontend) to appease the lint gods
yindo added the pull-request label 2026-02-21 20:46:26 -05:00
yindo closed this issue 2026-02-21 20:46:26 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify#29886