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

Locked pages view and options to hide remote management and show the commit author #112

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

morja
Copy link

@morja morja commented Sep 25, 2024

I added a locked pages view and extra options to hide remote management and show the commit author.

Maybe the locked pages could be their own plugin, but I like having this within the same menu section.

In my scenario, I only have a local git repository and I would like to have a history of edits and the info who did it.
Also sometimes other editors forget to save their changes, so the locked pages view is very helpful.

And btw. thanks a lot for the great plugin!

@thathoff
Copy link
Owner

Hi, thanks for your pull request! Would it be possible to split these changes into three separate commits? The author feature is already implemented (see #111), and from my point of view, hiding remote management and showing locked files are two different features.

I'm also not sure the locked files feature should be included in this plugin, as it has nothing to do with git. But I like the idea in general and I'm happy to think about including this feature.

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