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.

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

Adding an introduction to sftp tab, people who switch from SecureCRT to Tabby may want to use similar features of sftp tab. #8719

Merged
merged 1 commit into from
Aug 14, 2023

Conversation

wljince007
Copy link
Contributor

Adding an introduction to sftp tab, people who switch from SecureCRT to Tabby may want to use similar features of sftp tab.

@Eugeny
Copy link
Owner

Eugeny commented Jul 18, 2023

Heya I've just looked at your plugin code - good idea! However I would avoid placing profile templates into defaults since this will lead to confusing behaviour when a user already has existing profiles. A much cleaned solution would be to implement a ProfileProvider and provide your three templates (or automatically select one correct template depending on the platform!) as built-in profiles via getBuiltinProfiles like here: https://github.com/eugeny/tabby/blob/master/tabby-ssh/src/profiles.ts#L78C17-L98

@wljince007
Copy link
Contributor Author

Thank you for your patient guidance. I will try to modify the plugin code according to your guidance。

@wljince007 wljince007 closed this Jul 18, 2023
@wljince007
Copy link
Contributor Author

wljince007 commented Jul 23, 2023

@Eugeny The profile for the three platforms I provide here is a template. If it is provided as a builtin profile in the form of a ProfileProvider, users also need to copy and create a new profile to modify the sftp path. Once copied, the sftp tab plugin cannot know the profile name and cannot use this profile when creating sftp tab.

I see an isTemplate attribute in the profile. If this is set be true, the templates of the three platforms will not appear in the user's selection of "Profiles&Connections". However, the settings can be edited according to the local profile. This is currently the best implementation within my capabilities, and it is the simplest way for the sftp tab plugin to maintain, keep the minimum coupling between sftp-tab and tabby.

As a concept, functions of sftp-tab plugin is expected to be implemented by you by default, so that it can be used across the entire platform.

@wljince007 wljince007 reopened this Jul 23, 2023
@wljince007 wljince007 closed this Jul 24, 2023
@Eugeny
Copy link
Owner

Eugeny commented Jul 24, 2023

I think you're just a little confused by isTemplate. It should be perfect for your plugin because profiles with isTemplate: true will appear in the "Create new profile" selector as templates, allowing your users to quickly create SFTP profiles as needed.

@wljince007
Copy link
Contributor Author

wljince007 commented Jul 26, 2023

@Eugeny Actually, ssh2sftp_XXX_template is intended to be used as a template, the main goal is not allow user to create SFTP profiles, the initial idea was let sftp-tab plugin uses this template to create sftp connections of ssh automatic. The ssh2sftp_XXX_template are provided in the form of local profile template, just want to borrow the interface modification function of the localProfile in the tabby-local to let users modify the sftp path and save it in the config file.

Is this possible? Do I still need to develop my own pug to make interface modifications sftp path?

@wljince007 wljince007 reopened this Jul 30, 2023
@wljince007 wljince007 closed this Aug 2, 2023
@wljince007
Copy link
Contributor Author

@wljince007 wljince007 reopened this Aug 11, 2023
@Eugeny
Copy link
Owner

Eugeny commented Aug 11, 2023

LGTM but please make sure you're not making other unrelated changes to the READMEs

@wljince007 wljince007 reopened this Aug 14, 2023
@wljince007
Copy link
Contributor Author

Last time I used VSCode, it automatically modify the error of READES, I have used vim to redo the warranty and modify the sftp-tab introduction.

@Eugeny Eugeny merged commit 2262d59 into Eugeny:master Aug 14, 2023
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants