Skip to content

Project level Gerrit settings#480

Open
videogameaholic wants to merge 8 commits intouwolfer:intellij2020.3from
videogameaholic:intellij2020.3
Open

Project level Gerrit settings#480
videogameaholic wants to merge 8 commits intouwolfer:intellij2020.3from
videogameaholic:intellij2020.3

Conversation

@videogameaholic
Copy link
Copy Markdown

Project level Gerrit settings

  • Plugin settings are now project level. Seamlessly switch between projects housed on separate Gerrit servers.
  • Compatibility: Previously existing settings are loaded as the application level default.
  • When opening a project for the first time, application level default settings are used.
  • Added buttons for Save as Default Settings and Load Default Settings.

Settings are stored per project.  New projects load from the default settings.  Settings panel has buttons to save to and load from the default settings.
Fixed an issue with credentials in PasswordSafe.  Added a few null safety checks.
@videogameaholic
Copy link
Copy Markdown
Author

gerrit-intellij-plugin-1.2.6-203.zip
Attaching binary for testing

Copy link
Copy Markdown

@jjrauch32-lab jjrauch32-lab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okk

@uwolfer
Copy link
Copy Markdown
Owner

uwolfer commented Dec 19, 2025

@videogameaholic Thanks for your PR, and sorry for not jumping in earlier. I think this is a pretty interesting feature in some setups.

Compatibility: Previously existing settings are loaded as the application level default.

How much testing have you done here? When coming from an old version to this one, will it take previous global settings for any project in order to not break for existing clients? Do you see any risks here?

@videogameaholic
Copy link
Copy Markdown
Author

Existing settings should be loaded as the default so anyone upgrading should not notice a difference. We were using this at work in 2023, but we have since migrated to Azure DevOps rather than multiple on-prem Gerrit servers which is why I haven't given my branch any love recently. I did notice that the "Force default branch name" checkbox has been added since I last updated this branch and the UI needed updated for merge.

Copy link
Copy Markdown

@jjrauch32-lab jjrauch32-lab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok

Copy link
Copy Markdown

@jjrauch32-lab jjrauch32-lab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok

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.

4 participants