Skip to content

docs: update admin nav path and Add connector flow - #50

Merged
ggreer merged 1 commit into
mainfrom
docs/nav-and-create-connector-drawer
Sep 10, 2026
Merged

ggreer merged 1 commit into
mainfrom
docs/nav-and-create-connector-drawer

Conversation

@mindymo

@mindymo mindymo commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Summary

Two upstream product changes affect this connector's docs:

  • Admin nav reorg (ductone/c1#24648): the sidebar path is now Apps > Connectors, replacing Integrations > Connectors.
  • New Add connector drawer (ductone/c1#24933, ductone/c1#25052): the old 3-option "Add connector" flow (unmanaged app / managed app / new managed app + single owner) is replaced by a guided drawer: choose destination (new app / existing app) → optionally link to an application discovered from your identity provider → name/description → owners (multi-owner picker) → Add.

Both features are live in production. This PR brings the connector's own docs source in line with what's already merged in ConductorOne/docs (see ConductorOne/docs#527 and ConductorOne/docs#528), so the next sync doesn't reintroduce the stale flow.

Corrections were verified against the actual drawer code and applied directly to this file — not reapplied as a blind patch, since source docs can drift from what's synced into the docs site.

Test plan

  • Preview build looks correct

- Apps > Connectors replaces Integrations > Connectors in the admin
  nav (c1#24648).
- The Add connector walkthrough now reflects the new guided drawer:
  choose destination (new app / existing app) -> optional IdP link
  -> name/description -> owners -> Add (c1#24933, c1#25052).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

Connector PR Review: docs: update admin nav path and Add connector flow

Blocking Issues: 0 | Suggestions: 0 | Threads Resolved: 0
Criteria: Criteria status: loaded .claude/skills/ci-review.md from trusted base a8f945bc2c10.
Review mode: full
View review run

Review Summary

Scanned the full PR diff for security and correctness. This is a documentation-only change touching docs/connector.mdx (8 insertions, 11 deletions): it updates the C1 admin nav path from Integrations > Connectors to Apps > Connectors and replaces the old three-option "Add connector" flow with the new guided drawer steps (destination, optional IdP app link, name/description, owners, Add). No Go source, config, or dependency manifests changed, so there is no connector implementation, provisioning, pagination, or dependency surface in scope. The new nav wording is consistent with the already-correct Apps > Managed apps reference later in the same file (docs/connector.mdx:169), and the Steps/Step MDX tags remain balanced (10/10, 2/2). No prior findings existed on this PR and no new issues were found.

Security Issues

None found.

Correctness Issues

None found.

Suggestions

None.

@github-actions github-actions Bot 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.

No blocking issues found.

@ggreer
ggreer merged commit c665c53 into main Sep 10, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants