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

Feat: Automatically update MR summary details #427

Conversation

jakubbortlik
Copy link
Collaborator

@jakubbortlik jakubbortlik commented Nov 21, 2024

Hi @harrisoncramer, this is an implementation of #401. I've marked it as a draft because it will need some reworking if #426 is merged in. I'd be grateful if you could review it and consider including it in the plugin.

With this PR, when the Summary popup is open the following items in the Details are updated automatically when the user runs a command that changes the MR state:

  • Updated
  • Status
  • Assignees
  • Reviewers
  • Labels

@jakubbortlik jakubbortlik force-pushed the feat-automatically-update-mr-summary branch from 339669e to c995305 Compare December 6, 2024 22:32
@jakubbortlik jakubbortlik marked this pull request as ready for review December 6, 2024 22:55
@harrisoncramer harrisoncramer merged commit a3aa79a into harrisoncramer:develop Dec 8, 2024
6 checks passed
@jakubbortlik jakubbortlik deleted the feat-automatically-update-mr-summary branch December 9, 2024 07:42
harrisoncramer added a commit that referenced this pull request Dec 11, 2024
* Feat: Enable sorting discussions by original comment (#422)
* Feat: Improve popup UX (#426)
* Feat: Automatically update MR summary details (#427)
* Feat: Show update progress in winbar (#432)
* Feat: Abbreviate winbar (#439)
* Fix: Note Creation Bug (#441)
* Fix: Checking whether comment can be created (#434)
* Fix: Syntax in discussion tree (#433)
* fix: improve indication of resolved threads and drafts (#442)
* Docs: Various minor improvements (#445)

---------

Co-authored-by: Jakub F. Bortlík <[email protected]>
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