[New] Pull View: Delete Branch option after merge #860

Closed
opened 3 months ago by 6543 · 6 comments
6543 commented 3 months ago
Collaborator

Delete branch option as per webUI

check if branch with same head commit and name as branch of merged pull exist, if true show button

Delete branch option as per webUI check if branch with same head commit and name as branch of merged pull exist, if true show button
6543 added the
Feature
label 3 months ago
Owner

Delete branch is already there if I understand you correctly.

Delete branch is already there if I understand you correctly.
Poster
Collaborator

@mmarif not it is about deleting a branch after it got merged

@mmarif not it is about deleting a branch **after** it got merged
mmarif changed title from [New] Pull View: Delete Branch option to [New] Pull View: Delete Branch option after merge 2 months ago

I could do this, should I send a pr?

I could do this, should I send a pr?
Owner

Sure, but before you spent time on it. Can you detail out how you will implement it UI wise in the app?

Sure, but before you spent time on it. Can you detail out how you will implement it UI wise in the app?

I would put it into the BottomSheet dialog, just like the options to merge the pr, close it etc. (but only show it if the pr is merged/closed)

I would put it into the BottomSheet dialog, just like the options to merge the pr, close it etc. (but only show it if the pr is merged/closed)
Owner

Nice, I am fine with that implementation.

Nice, I am fine with that implementation.
mmarif added this to the 4.0.0 milestone 1 week ago
mmarif closed this issue 4 days ago
mmarif closed this issue 4 days ago
Sign in to join this conversation.
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.