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

[FR] AutoCheckout on SAVE #189

Open
linqen opened this issue Nov 8, 2024 · 5 comments
Open

[FR] AutoCheckout on SAVE #189

linqen opened this issue Nov 8, 2024 · 5 comments

Comments

@linqen
Copy link

linqen commented Nov 8, 2024

Hi, first of all, thank you so, so much for your work here. I have been using your plugin for a while and, without it, we won't be able to have the great workflow we have right now!

I wanted to request a feature, or maybe to ask some directions so I can implement it and make a PR.

Right now there is an option (this is offered by the engine AFAIK) to auto-checkout when a file is modified. This is too much for a team I think because if some of the users are not very technical, project can get crazy too fast with everything locked.

My idea is to have an auto-checkout but when you click save instead of modification. This would remove the extra step of clicking CHECKOUT on files that can be checked out. If the file is already checked out by someone else, the window should appear and offer you the chance to make it writeable, or cancel the saving process.

Thank you!

@mastercoms
Copy link
Member

Hi, thanks for the suggestion. This is a great idea but I feel like this would be better offered in the engine rather than in this plugin.

@linqen
Copy link
Author

linqen commented Nov 8, 2024

Hi, thanks for the suggestion. This is a great idea but I feel like this would be better offered in the engine rather than in this plugin.

Do you think that is possible technically to handle this from plugin side? If so, I could take a look and try a PR.

In our project, would be a huge overkill to switch to a built engine just for this, so if possible from plugin side, I would go that route.

@mastercoms
Copy link
Member

I do think it's possible, but I probably wouldn't include it in the main branch. You can do this in a fork if you'd like.

@linqen
Copy link
Author

linqen commented Nov 8, 2024

Great, I will, thank you! Any advice where I could take a look for this?

@mastercoms
Copy link
Member

I'm not really so sure, sorry

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

No branches or pull requests

2 participants