-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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 Issue #4946 Basic folder upload for sftp electron. #9891
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few comments, looks good otherwise!
Also, startUploadFromDragEvent
would need to also support folder drop events too
Co-authored-by: Eugene <[email protected]>
What do you think about making a directory as an |
If you'd like to implement empty folder "uploading", I suggest two ways: |
Oaky I'v done all the feats. |
cc @Eugeny you can now have a final check for this pr.Thx |
Looking good now, thanks! @all-contributors please add @marko1616 for code |
We had trouble processing your request. Please try again later. |
🚀 Basic Folder Upload Feature for SFTP in Electron
📝 Description
I have implemented a folder upload feature for the SFTP functionality within Electron. This enhancement allows users to upload entire folders, which could significantly improve the user experience for bulk file transfers.
🙏 Acknowledgments
I would like to express my sincere gratitude to all the project developers for their continued efforts and invaluable contributions. Your hard work has laid the foundation for features like this to be developed.
🚧 Next Steps
If the team agrees that this is a suitable implementation:
startUploadFromDragEvent
.Relative issues
#4946 #9841 #9738 #9569
Thank you for taking the time to review this PR. Your insights and suggestions will be highly appreciated!