Skip to content
This repository has been archived by the owner on Nov 22, 2022. It is now read-only.

Allow 'issue' and 'mr' update to unlock discussion #400

Closed
maxice8 opened this issue Dec 23, 2020 · 2 comments · Fixed by #401
Closed

Allow 'issue' and 'mr' update to unlock discussion #400

maxice8 opened this issue Dec 23, 2020 · 2 comments · Fixed by #401
Labels
cmd: issue Related to issues management cmd: mr Issue is related to merge request management enhancement New feature or request

Comments

@maxice8
Copy link
Collaborator

maxice8 commented Dec 23, 2020

Describe the feature or problem you'd like to solve
There is currently no way to unlock a previously locked discussion, there is --lock-discussion but no equivalent for unlocking it.

Propose a Solution
Add a --unlock-discussion flag to mr update and issue update
@maxice8 maxice8 added enhancement New feature or request cmd: issue Related to issues management cmd: label Releated to label command labels Dec 23, 2020
@maxice8
Copy link
Collaborator Author

maxice8 commented Dec 23, 2020

If a yes is give I'll work on it

@maxice8 maxice8 added cmd: mr Issue is related to merge request management and removed cmd: label Releated to label command labels Dec 23, 2020
@profclems
Copy link
Owner

Nice. You can go ahead.

I was also thinking of using the --lock-discussion. If set to false, unlocks the discussion.
But looking much into it, adding --unlock-discussion is expressive but both shouldn't be used at the same time

@maxice8 maxice8 mentioned this issue Dec 23, 2020
Merged
5 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
cmd: issue Related to issues management cmd: mr Issue is related to merge request management enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Unlock discussion
2 participants
@maxice8 @profclems