docs icon indicating copy to clipboard operation
docs copied to clipboard

Parameters missing from GitHub App Manifests

Open blakegearin opened this issue 2 years ago • 3 comments

Code of Conduct

What article on docs.github.com is affected?

https://docs.github.com/en/[email protected]/developers/apps/building-github-apps/creating-a-github-app-from-a-manifest#github-app-manifest-parameters

What changes are you suggesting?

The "GitHub App Manifest parameters" section should be updated to include all possible parameters.

I haven't tried myself, but it seems likely that request_oauth_on_install and setup_on_update are supported and undocumented manifest parameters.

Request to open this issue: https://github.com/github/docs/issues/21659#issuecomment-1307841708

Additional information

Here's a comparison between the parameters listed in the manifest documentation versus the URL parameters documentation.

Parameter URL parameters Manifest parameters
name
description
url
callback_urls
public
request_oauth_on_install
setup_url
setup_on_update
webhook_active ❌ see: hook_attributes
webhook_url ❌ see: hook_attributes
hook_attributes
events ❌ see: default_events
default_events ❌ see: events
single_file_name
single_file_paths
redirect_url
default_permissions ❌ passed in as key-value pairs

blakegearin avatar Nov 28 '22 06:11 blakegearin

:point_up: yes please!

AgustinRamiroDiaz avatar Nov 28 '22 17:11 AgustinRamiroDiaz

@blakegearin Thanks so much for opening an issue! I'll triage this for the team to take a look :eyes:

cmwilson21 avatar Nov 28 '22 21:11 cmwilson21

Thanks @blakegearin , I can confirm both the request_oauth_on_install and setup_on_update are supported parameters for GitHub App manifests. You or anyone else is welcome to open a PR to add these parameters to the "GitHub App Manifest parameters" table as outlined below.

request_oauth_on_install | boolean | Set to true to request the user to authorize the GitHub App, after the GitHub App is installed. setup_on_update | boolean | Set to true to redirect users to the setup_url after they update your GitHub App installation.

vgrl avatar Nov 30 '22 01:11 vgrl

Hi @cmwilson21 , Olayinka here! I started work on the project with a couple of issues from the project board. Made change to address this issue and raised a PR for it here - https://github.com/github/docs/pull/23736. Please check it out when chanced!

OlayinkaAtobiloye avatar Feb 06 '23 20:02 OlayinkaAtobiloye