-
Notifications
You must be signed in to change notification settings - Fork 60
Add Multi Org Support Section for GitHub Ocean #2885
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
This pull request is automatically being deployed by Amplify Hosting (learn more). |
- **With classic PAT**: Specify a list of organizations: `githubOrganizations: ["org1", "org2", "org3"]` | ||
- **With classic PAT**: Leave empty to sync all organizations the PAT user is a member of: `githubOrganizations: []` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DRY
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update the description with the link to the document in preview mode following the description template
:::warning Breaking change in v2.0.0-beta | ||
The configuration parameter has changed from `githubOrganization` (singular) to `githubOrganizations` (plural, array format). You must update your configuration when upgrading to v2.0.0-beta. | ||
|
||
**Before:** | ||
```yaml | ||
githubOrganization: "my-org" | ||
``` | ||
**After (v2.0.0-beta):** | ||
```yaml | ||
githubOrganizations: ["my-org"] # Now an array | ||
``` | ||
::: | ||
:::info Authentication requirements | ||
- **Classic PAT**: Supports multiple organizations | ||
- **Fine-grained PAT**: Does NOT support multi-organization | ||
- **GitHub App**: Supports exactly **one organization** only | ||
::: | ||
:::caution Performance impact | ||
Syncing multiple organizations increases API calls and may slow down the integration. Consider syncing only the organizations you need. | ||
::: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't use two or more admonitions in a row
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Find another way to communicate this
:::info Authentication requirements | ||
- **Classic PAT**: Supports multiple organizations. Fine-grained PAT tokens do not support multi-organization authentication. | ||
- **GitHub App**: Only supports a **single organization** (minimum 1, maximum 1). You must specify exactly one organization in the `githubOrganizations` array. | ||
::: | ||
|
||
:::caution Performance impact | ||
Syncing multiple organizations will increase the number of API calls to GitHub and may slow down the integration. The more organizations you sync, the longer the resync time and the higher the API rate limit consumption. Consider syncing only the organizations you need. | ||
::: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same comment about the admonition
:::warning Authentication limitations | ||
- **Classic PAT**: Supports multiple organizations. Fine-grained PAT tokens do not support multi-organization authentication. | ||
- **GitHub App**: Only supports a **single organization** (minimum 1, maximum 1). You must specify exactly one organization in the `githubOrganizations` array. | ||
::: | ||
|
||
:::caution Performance impact | ||
Syncing multiple organizations will increase the number of API calls to GitHub and may slow down the integration. The more organizations you sync, the longer the resync time and the higher the API rate limit consumption. Consider syncing only the organizations you need. | ||
::: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Added docs pages
Please also include the path for the added docs
/
)/platform-overview/port-components/blueprint
)Updated docs pages
Please also include the path for the updated docs
/
)/platform-overview/port-components/blueprint
)