Issue #2683: Hitting the PR "merge" button should also submit any comment text in the comment text box - pagure

pagure

#2683 Hitting the PR "merge" button should also submit any comment text in the comment text box

Closed: Fixed 5 years ago Opened 6 years ago by mattdm.

Right now, if you write a comment but don't press "Update Pull-Request" to submit it, and then hit Merge, the comment just gets lost. It's nice to be able to comment as Merging, in one step. ("Thanks! Looks good to me!")

Metadata Update from @pingou:
- Issue tagged with: RFE

6 years ago

Metadata Update from @ryanlerch:
- Issue set to the milestone: 5.2

5 years ago

Metadata Update from @pingou:
- Issue assigned to pingou

5 years ago

pingou commented 5 years ago

Commit 8821f71 fixes this issue

Metadata

Assignee

pingou

Tags

Blocking

None

Depending on

None

Priority

None

Milestone

5.2

Related Pull Requests

#4035 Merged 5 years ago

pagure

Source Code

Documentation

#2683 Hitting the PR "merge" button should also submit any comment text in the comment text box Closed: Fixed 5 years ago Opened 6 years ago by mattdm.

Metadata

RFE

Related Pull Requests

#2683 Hitting the PR "merge" button should also submit any comment text in the comment text box

Closed: Fixed 5 years ago Opened 6 years ago by mattdm.