-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Open jira settings when not configured #7033
Open jira settings when not configured #7033
Conversation
@@ -62,8 +62,9 @@ public JiraCommitHintPlugin() | |||
|
|||
public override bool Execute(GitUIEventArgs args) | |||
{ | |||
if (_enabledSettings.ValueOrDefault(Settings)) | |||
if (!_enabledSettings.ValueOrDefault(Settings)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this work at all now?
Could you please explain the use case, it is not clear to me what was wrong that we're fixing here. |
It's just a little fix and improvement. Before this PR, we had a I discovered, when developing the same plugin for Azure DevOps that a feature was written to show a message box with an example of a commit message by querying Jira with all the parameters (not a great value but why not). The first commit fix the condition to enable this feature. And while I was there, I did the same thing that we do on some other plugin , that is, to display the settings window when the plugin is not configured. This is the changed introduced in the 2nd commit. Even if you don't have Jira (I don't, I just backport from the AzureDevops plugin), you could test this part.
@gerhardol So, yes, it works now 😉 |
8747493
to
37dcb5c
Compare
Merge if @gerhardol is happy |
37dcb5c
to
49f5083
Compare
So, I let @gerhardol merge it 😉 |
Proposed changes
Test methodology
Test environment(s)