docs: document GitHub App credentials and manifest flow#1302
Open
swampie wants to merge 25 commits intoenterprise-26.1-documentationfrom
Open
docs: document GitHub App credentials and manifest flow#1302swampie wants to merge 25 commits intoenterprise-26.1-documentationfrom
swampie wants to merge 25 commits intoenterprise-26.1-documentationfrom
Conversation
Add a new GitHub App subsection to the Git integration overview for both cloud and enterprise docs, covering the credentials form (Access token vs GitHub App, with Create-and-add vs Add-preexisting), the manifest-based app creation flow, repository installation steps, and how to handle duplicate credentials. Includes the credentials form screenshot for the new GitHub App tab. Co-Authored-By: Claude Opus 4.6 <[email protected]>
✅ Deploy Preview for seqera-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Replace the second image placeholder with a screenshot of GitHub's "Create GitHub App" page showing the pre-filled manifest, in both cloud and enterprise docs. Co-Authored-By: Claude Opus 4.6 <[email protected]>
Replace the third image placeholder with a screenshot of GitHub's App installation page showing repository selection, in both cloud and enterprise docs. Co-Authored-By: Claude Opus 4.6 <[email protected]>
Member
christopher-hakkaart
left a comment
christopher-hakkaart
left a comment
There was a problem hiding this comment.
This looks really good. Can you confirm you are happy with these suggestions?
TLDR;
- Making text active voice present tense where possible
- Splitting some steps into sub-points to improve flow
- Some rewording
- Em-dash into
:or.
Please note that this will be in the cloud release now but will not appear in the enterprise docs until the next release. If this is being added to the current Enterprise release we will need to add it to the versioned folder.
Also, I'm happy to migrate the suggestions to the other version(s) once they are confirmed.
Member
Author
|
I am happy with all your suggestions @christopher-hakkaart : just wanted to kick off something for EDU team. It will be in cloud soon, I'll probably open next week after some testing in staging and yes it needs to be versioned for 26.1. Thanks for helping: we can leave this open until GA release on cloud. |
Member
|
Awesome. I will apply the suggestions and mark as do not merge. |
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
Co-authored-by: Chris Hakkaart <[email protected]> Signed-off-by: Justine Geffen <[email protected]>
justinegeffen
changed the base branch from
master
to
enterprise-26.1-documentation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add a new GitHub App subsection to the Git integration overview for both cloud and enterprise docs, covering the credentials form (Access token vs GitHub App, with Create-and-add vs Add-preexisting), the manifest-based app creation flow, repository installation steps, and how to handle duplicate credentials. Includes the credentials form screenshot for the new GitHub App tab.