feat(onboarding): add maintainer completion screen#412
Open
Prateeks16 wants to merge 2 commits into
Open
Conversation
Step 3 of the maintainer onboarding wizard (Coder-s-OG-s#332). Shows a '<account> is live on MergeShip' headline and a summary card with three rows reflecting what was just configured: - Repos connected: rotating ticker (Coder-s-OG-s#334) over the managed repos - AI-generated PR detection: placeholder until Coder-s-OG-s#329 lands its setting - Mentor chain: reads installation_settings.auto_assign_mentor_chain Plus primary 'Go to dashboard' and secondary 'Adjust repos' actions. Reuses the existing getRepoPicker / getInstallationSettings server actions.
Contributor
|
@Prateeks16 is attempting to deploy a commit to the codersogs-3057's projects Team on Vercel. A member of the Team first needs to authorize it. |
Ayush-Patel-56
requested changes
Jun 24, 2026
Addresses review on Coder-s-OG-s#412: throw on getRepoPicker / getInstallationSettings failure instead of swallowing it into 'no repos connected' / 'Off' right after the maintainer configured these in step 2. Mirrors /onboarding/repos.
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
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.
Summary
Maintainer onboarding step 3 — the completion screen (#332). After picking repos (step 2, #328) the maintainer lands here: a " is live on MergeShip" headline and a summary card reflecting what they just configured.
Type of Change
Related Issue
Closes #332
What was changed?
src/app/onboarding/complete/page.tsx(server component) — resolves the maintainer's first install, then reads managed repos + saved settings in parallel.RepoNameTicker(Completion screen: rotating repo name display for multiple repos #334, already onmain) over themanagedrepos, with a count badge.installation_settings.auto_assign_mentor_chain(On → "routing L0/L1 to senior maintainers" / Off)./maintainer) and secondary "Adjust repos" (/onboarding/repos) actions.Notes
getRepoPicker/getInstallationSettingsserver actions — no new actions, schema, or migration./onboarding/analyze,/onboarding/repos).Test plan
make test— 534 passing (ticker helpers inrepo-name-ticker.test.tsalready cover the rotating logic).make typecheck,make lint, prettier — clean./onboarding/completeas a maintainer persona — headline shows the org, ticker cycles when >1 managed repo, mentor-chain row reflects the toggle, buttons route correctly.Checklist