Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

github app: toggle app default visibility#53286

Merged
courier-new merged 1 commit into
batches/commit-signingfrom
kr/public-checkbox
Jun 12, 2023
Merged

github app: toggle app default visibility#53286
courier-new merged 1 commit into
batches/commit-signingfrom
kr/public-checkbox

Conversation

@courier-new

@courier-new courier-new commented Jun 10, 2023

Copy link
Copy Markdown
Contributor

Adds a <Checkbox /> to the Create GitHub App form for toggling the default visibility of the App from private to public. This will populate the value in the App manifest.

I also added a short explanation about GitHub App installations and the ability to install them in multiple places if the app is public vs. private to the main GitHub App page, with a link the part of our docs that describes it in more detail.

Demo

Screen.Recording.2023-06-09.at.9.46.40.PM.mov

And a screenshot from the normal Repos create flow:

image

Test plan

Manual testing.

@sourcegraph-bot

sourcegraph-bot commented Jun 10, 2023

Copy link
Copy Markdown
Contributor

📖 Storybook live preview

@BolajiOlajide BolajiOlajide left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pjlast pjlast left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this!

Base automatically changed from kr/ux-create-app-2 to batches/commit-signing June 12, 2023 15:29

@st0nebreaker st0nebreaker left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this! And the clear instructions/descriptions!

@courier-new courier-new merged commit 56913fb into batches/commit-signing Jun 12, 2023
@courier-new courier-new deleted the kr/public-checkbox branch June 12, 2023 15:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

batch-changes Issues related to Batch Changes cla-signed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants