fix(FTP Node): Use filename instead of remote filepath for downloaded binary data #6170
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Great PR! Please pay attention to the following items before merging: Files matching
Files matching
Files matching
Make sure to check off this list before asking for review. |
n8n-assistant
bot
added
n8n team
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #6170 +/- ##
==========================================
- Coverage 20.08% 20.06% -0.02%
==========================================
Files 2626 2622 -4
Lines 118218 118083 -135
Branches 18572 18556 -16
==========================================
- Hits 23740 23691 -49
+ Misses 93758 93677 -81
+ Partials 720 715 -5
☔ View full report in Codecov by Sentry. |
maspio
approved these changes
May 3, 2023
|
|
netroy
force-pushed
the
NODE-491-fix-binary-fileName-in-ftp-sftp
branch
from
5cee450 to
06b3267
Compare
maspio
approved these changes
May 4, 2023
|
|
1 similar comment
|
|
netroy
force-pushed
the
NODE-491-fix-binary-fileName-in-ftp-sftp
branch
from
06b3267 to
a3a96cd
Compare
maspio
approved these changes
May 5, 2023
|
✅ All Cypress E2E specs passed |
MiloradFilipovic
added a commit
that referenced
this pull request
May 5, 2023
* master: fix(FTP Node): Use filename instead of remote filepath for downloaded binary data (#6170) ci: Fix test database cleanup (no-changelog) (#6188) refactor(core): Delete boilerplate code across migrations (no-changelog) (#5254) refactor(editor): Add infix to Pinia stores (no-changelog) (#6149) ci: Fix linting issues on master (no-changelog) (#6186)
MiloradFilipovic
added a commit
that referenced
this pull request
May 9, 2023
* master: (31 commits) feat(Date & Time Node): Overhaul of the node (#5904) refactor: Add deprecation notice for WEBHOOK_TUNNEL_URL (#6194) refactor: Add deprecation notice for WEBHOOK_TUNNEL_URL (#6194) feat(HubSpot Node): Overhaul the HubSpot Node (#4337) ci: Create a nightly v1 docker image (no-changelog) (#6197) refactor(core): Add deprecation notice for own mode (#6195) feat(core): Update config defaults for for v1 (no-changelog) (#6196) test(Read Binary Files Node): Unit tests (no-changelog) (#5459) refactor: Add deprecation notice for MySQL and MariaDB (#6189) test: Create custom jest error messages using jest-expect-message (no-changelog) (#5666) fix(core): Move nodeExecute InternalHook calls to hookFunctionsSave (#6193) fix(FTP Node): Use filename instead of remote filepath for downloaded binary data (#6170) ci: Fix test database cleanup (no-changelog) (#6188) refactor(core): Delete boilerplate code across migrations (no-changelog) (#5254) refactor(editor): Add infix to Pinia stores (no-changelog) (#6149) ci: Fix linting issues on master (no-changelog) (#6186) fix(editor): Update and add design system checkbox component to Editor (#6178) fix(editor): Display SSO entry in Settings on Cloud (#6181) feat(Code Node): Add Python support (#4295) fix(editor): Update and fix storybook (was failing to run in local dev mode) (#6180) ... # Conflicts: # packages/editor-ui/src/stores/nodeTypes.store.ts # packages/nodes-base/nodes/Postgres/Postgres.node.ts # packages/nodes-base/nodes/Postgres/v2/actions/versionDescription.ts
MiloradFilipovic
added a commit
that referenced
this pull request
May 9, 2023
…rce-mapper-ui-P2 * feature/resource-mapping-component: (31 commits) feat(Date & Time Node): Overhaul of the node (#5904) refactor: Add deprecation notice for WEBHOOK_TUNNEL_URL (#6194) refactor: Add deprecation notice for WEBHOOK_TUNNEL_URL (#6194) feat(HubSpot Node): Overhaul the HubSpot Node (#4337) ci: Create a nightly v1 docker image (no-changelog) (#6197) refactor(core): Add deprecation notice for own mode (#6195) feat(core): Update config defaults for for v1 (no-changelog) (#6196) test(Read Binary Files Node): Unit tests (no-changelog) (#5459) refactor: Add deprecation notice for MySQL and MariaDB (#6189) test: Create custom jest error messages using jest-expect-message (no-changelog) (#5666) fix(core): Move nodeExecute InternalHook calls to hookFunctionsSave (#6193) fix(FTP Node): Use filename instead of remote filepath for downloaded binary data (#6170) ci: Fix test database cleanup (no-changelog) (#6188) refactor(core): Delete boilerplate code across migrations (no-changelog) (#5254) refactor(editor): Add infix to Pinia stores (no-changelog) (#6149) ci: Fix linting issues on master (no-changelog) (#6186) fix(editor): Update and add design system checkbox component to Editor (#6178) fix(editor): Display SSO entry in Settings on Cloud (#6181) feat(Code Node): Add Python support (#4295) fix(editor): Update and fix storybook (was failing to run in local dev mode) (#6180) ... # Conflicts: # packages/editor-ui/src/stores/nodeTypes.store.ts
netroy
added a commit
that referenced
this pull request
May 15, 2023
|
Got released with |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
NODE-491
Fixes: #6026