Githawk: Dis/Approve pull requests

Created on 22 Aug 2018  ·  12Comments  ·  Source: GitHawkApp/GitHawk

would be nice to be able to also approve or disapprove prs. I can do everything except for that.


Bug Report Dump (Auto-generated)

Version 1.22.0 (1534395229)
Device: iPhone X (iOS 11.4.1)
TestFlight: false

🆕 feature request

Most helpful comment

I barely I started with playing with the code. The task is quite daunting when you consider all the cases. I’ll hopefully have time to look at it soon

Sent with GitHawk

All 12 comments

Aaaaaaabsolutely. The next update I want to get these local pushes working, then focus on a PR-rich update.

Sent with GitHawk

@rnystrom I will have some time to jump on that. I recenlty changed my github workflow to 100no emails :)

yey

@rnystrom did you have anything in mind for the positioning of such actions?

I am futzing with multiple options:

  • From the menu (with labels and milestones) add Approve / Reject
  • From the menu add 'Add Review' which pushes a new controller with a textView + MergeButton (customized)
  • From the files list

etc...

What is the status of this feature? If I can weight in on the location, it would be really cool to be able to see the “Add review” button in place of the “Not authorized to merge” when there is a missing but required review. Additionally I think it would be natural to add this to the files view to keep it consistent with github’s process.

I barely I started with playing with the code. The task is quite daunting when you consider all the cases. I’ll hopefully have time to look at it soon

Sent with GitHawk

Woah! This would be awesome 👏

Sent with GitHawk

is this feature request still planned?

Approving a PR was the first task I needed to do when downloading and opening githawk a few minutes ago. Looks like a great app, but the lack of the ability to do PR review has me downloading another app.

@leecalcote also given I requested it in 2018 and we're in 2020 now. :(

Bump, as a open source project maintainer this feature is greatly needed

I won’t have time to work on this, unassigning myself

Was this page helpful?
0 / 5 - 0 ratings

Related issues

rnystrom picture rnystrom  ·  3Comments

BasThomas picture BasThomas  ·  3Comments

BasThomas picture BasThomas  ·  3Comments

rnystrom picture rnystrom  ·  3Comments

rnystrom picture rnystrom  ·  3Comments