feature(frontend): SFTP#528
Draft
jasonlessenich wants to merge 52 commits into
Draft
Conversation
- implement selection - make isHidden a store key
- first pass on implementing file writing - handle symlinks
- dart format
- dart format
…tend/feature/sftp # Conflicts: # frontend/lib/modules/session/view/sftp_session_page.dart
- move download logic to isolate and implement upload & transfer - always spawn a new sftp session with each isolate - first working version of dragging files to another sftp view
- replace int index with unique string file ids - allow navigating when downloading a file
- re-implement uploading modified files - fix lint - add more todos :'(
- add missing addQueuedFile to file transfer
- also refresh directory when transfer/copy is done
- add todo
- only rename the file if dragged to the same host
- add downloaded size - minor cleanup
- rework FileData - use SftpTransferType to determine queue icon
- allow deleting items - fix comment - add variant to custom context menu
- add Connect via SFTP to connection context menu
- cleanup temp file when deleting file
- dart format
- Allow to rename items - remove custom FocusableActionDetector (for now) - Add rowWrapper for context menu
# Conflicts: # frontend/lib/modules/session/view/session_page.dart
- fix breaking changes from forui 0.23
- use forui's FContextMenu in CustomContextMenu - dont allow 'opening' directories
- add separate directory open action
- further improve renaming
- move queue to navigation shell
- rename transfer queue to file transfer provider - further decoupled transfer logic - correctly kill isolate & delete file when cancelling - fix remoteToRemote transfers source & destination being flipped - correctly await remoteFile write on remoteToRemote transfers - add confirmation dialog for when attempting to overwrite an file on the remote host
- correctly implement back & forward navigation
- dart format
- fix top bar min height to avoid shifts when session is opened/closed - dont throw exception for invalid sessions
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
TODO: