-
Notifications
You must be signed in to change notification settings - Fork 12.4k
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.
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
fix(Wise Node): Use ISO formatting for timestamps #10288
Conversation
for posterity, can you please link to Wise docs that say that these timestamps need to be at UTC? 🙏🏽 |
Here's the Wise docs link. They don't explicitly say it's UTC only, but the format Regardless of the timezone part, the current date format is not supported. Since this functionality has worked in the past, I suspect a Moment.js update changed the default formatting. |
Only a friendly reminder. Sorry for being so persistent @Joffcom, I'm sure you have a lot to do, but without this fix I can't complete a pending accounting integration with Wise. Thank you for all the great work. |
Got released with |
Summary
Using Wise node's get many transfers functionality generates a 422 Unprocessable Entity error from the Wise API due to the date fields being formatted as
2024-07-02T15:10:15+00:00
instead of2024-07-02T15:10:15Z
. This PR fixes this issue.Related Linear tickets, Github issues, and Community forum posts
I've not opened an issue for this. I had to solve this issue for our internally deployed n8n and wanted to contribute back.
Review / Merge checklist
release/backport
(if the PR is an urgent fix that needs to be backported)