Skip to content

Commit 61d3cea

Browse files
authored
fix(github-checks): describe the connect flow that actually happens (#98)
The page promised repository selection on every Connect. That stopped being true when the Connect button moved to the OAuth authorize URL (dcd d122ccfe): an existing installation is now reconnected without a reinstall, which means without GitHub's repo picker. Users following the docs went looking for a step that never came. Describes both paths, points at the new Manage repositories link on the Integrations card as the way to change scope afterwards, and stops promising that disconnect always removes the App from GitHub — it's best-effort, and the console now says so when it can't. Adds the GitHub 404 on Connect to the troubleshooting list. It's a configuration problem on our side and nothing a customer can act on (devicecloud-dev/dcd#1140).
1 parent 6e25f0e commit 61d3cea

1 file changed

Lines changed: 18 additions & 4 deletions

File tree

‎ci-cd/github-checks.md‎

Lines changed: 18 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,14 +12,25 @@ The Action submits your tests and either polls for results or if you set `async:
1212

1313
## Connect the App
1414

15-
Open **Settings → Integrations** in the [console](https://console.devicecloud.dev/settings) and click **Connect GitHub**. GitHub walks you through installing the App and choosing which repositories it can see; when you come back, it's connected to your current team.
15+
Open **Settings → Integrations** in the [console](https://console.devicecloud.dev/settings) and click **Connect GitHub**. GitHub asks you to authorize DeviceCloud, and what happens next depends on whether you already have the App:
16+
17+
- **First time.** GitHub sends you on to install the App and choose which repositories it can see. Once you're done you land back on the Integrations page, connected to your current team.
18+
- **Already installed** (on another team, or you removed the connection earlier). We reconnect to your existing installation and return you straight to the console — no reinstall, and no repository step. Your existing repository selection carries over; change it any time with **Manage repositories on GitHub** on the Integrations card.
19+
20+
If you have the App installed on more than one GitHub account or organization, we'll ask which one to connect.
1621

1722
A few things worth knowing:
1823

1924
- Only owners and admins can connect GitHub for a team.
20-
- Your GitHub account needs to be linked to DeviceCloud first, under **Settings → Account**.
25+
- Your GitHub account needs to be linked to DeviceCloud first, under **Settings → Account**. This is separate from installing the App — it's how we know which DeviceCloud account an installation belongs to.
2126
- If you're an admin on more than one team, you may be asked which one to connect if we can't figure it out automatically using account context.
2227

28+
## Choosing which repositories
29+
30+
Checks only post on repositories the App can access. **Settings → Integrations** shows the current scope — either *All repositories* or a count of selected ones — with a **Manage repositories on GitHub** link that opens the installation's settings, where you can add or remove repositories.
31+
32+
Changes take effect immediately; we track GitHub's repository events, so the console reflects the new scope without you reconnecting.
33+
2334
## Run your tests
2435

2536
Run the [Action](github-actions.md) on your pull requests with `async: true`. It attaches the branch, commit, and PR number for you, which is how DeviceCloud knows where to post the check, so ensure you do not disable [`include-github-context`](github-actions.md#github--pr-context) (it's enabled by default).
@@ -56,13 +67,16 @@ The pass/fail count and total runtime. Below that, a table of every flow with it
5667
5768
## Managing the connection
5869
59-
**Settings → Integrations** shows the connected account and the repositories the App can reach. Owners and admins can disconnect from there, which removes the App from GitHub and stops the checks. If you reconnect later, DeviceCloud picks up your existing installation instead of making you install again. You can also remove the App from your GitHub settings.
70+
**Settings → Integrations** shows the connected account and the repositories the App can reach. Owners and admins can disconnect from there, which always removes the connection and stops the checks. We also try to uninstall the App from GitHub as part of that — if we can't, the console says so and links you to remove it yourself. You can uninstall from your GitHub settings at any time regardless.
71+
72+
If you reconnect later, DeviceCloud picks up your existing installation instead of making you install again.
6073
6174
## When a check doesn't show up
6275
6376
Usually it's one of these:
6477
65-
- **Nothing posts on the PR.** The App isn't installed on that repo, isn't connected to your team, or the run used a different team's API key. Check **Settings → Integrations**.
78+
- **Nothing posts on the PR.** The App isn't installed on that repo, isn't connected to your team, or the run used a different team's API key. Check **Settings → Integrations**, and confirm the repo is in scope under **Manage repositories on GitHub**.
79+
- **Connect GitHub lands on a GitHub 404.** That's a configuration problem on our side, not yours — nothing you can change will fix it. Get in touch and we'll sort it.
6680
- **The check is stuck in progress.** The run probably hasn't finished yet. Look it up in the console; the check only resolves once every flow reaches a final state.
6781
- **"That GitHub account isn't linked."** Link your GitHub identity under **Settings → Account**, then connect.
6882
- **The check isn't in the branch-protection list.** It has to run once on the repo before GitHub will offer it. Open a pull request first.

0 commit comments

Comments
 (0)