Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

When there is a decimal point in the node name, the output data is empty #7896

Closed
ybom77 opened this issue Dec 1, 2023 · 2 comments
Closed

When there is a decimal point in the node name, the output data is empty #7896

ybom77 opened this issue Dec 1, 2023 · 2 comments
Labels
in linear Issue or PR has been created in Linear for internal review Released

Comments

@ybom77
Copy link

ybom77 commented Dec 1, 2023

Describe the bug
When there is a decimal point in the node name, the output data is empty

To Reproduce
Steps to reproduce the behavior:

before :
image

after rename:
image

Expected behavior

the output data not empty

  • n8n Version [e.g. 1.18.0]
@Joffcom
Copy link
Member

Joffcom commented Dec 1, 2023

Hey @ybom77,

Thanks for the report, We are already tracking this internally as ADO-1244. I will add this to the list of open issues for it.

@Joffcom Joffcom added the in linear Issue or PR has been created in Linear for internal review label Dec 1, 2023
@janober janober mentioned this issue Dec 13, 2023
Merged
3 tasks
Joffcom pushed a commit that referenced this issue Dec 13, 2023
@janober @alexgrozav
fix(editor): Fix bug with node names with certain characters (#8013)
26f0d57 
## Summary
Fixes the issue that dots and other special characters can not be used
in node-names


## Related tickets and issues

https://linear.app/n8n/issue/ADO-1244/nodes-wont-output-data-if-the-name-contains-some-characters

https://community.n8n.io/t/bug-report-name-of-node-affects-output-data/31674
https://community.n8n.io/t/stripe-output-data/32418/1
https://community.n8n.io/t/monday-com-returning-items-but-no-data/31834
https://community.n8n.io/t/no-data-while-there-is-actually-data/32563
https://community.n8n.io/t/bug-report-periods-in-node-names/34119
#7896


## Review / Merge checklist
- [ ] PR title and summary are descriptive. **Remember, the title
automatically goes into the changelog. Use `(no-changelog)` otherwise.**
([conventions](https://github.com/n8n-io/n8n/blob/master/.github/pull_request_title_conventions.md))
- [ ] [Docs updated](https://github.com/n8n-io/n8n-docs) 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.

---------

Co-authored-by: Alex Grozav <alex@grozav.com>
@janober
Copy link
Member

janober commented Dec 13, 2023

Fix got released with n8n@1.21.0

@janober janober closed this as completed Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
in linear Issue or PR has been created in Linear for internal review Released
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Joffcom @janober @ybom77