Skip to content

refactor(react): restructure sources list and sources-add container - #220

Merged
RhysSullivan merged 1 commit into
rs/source-forms-04-other-formsfrom
rs/source-forms-05-sources-page
Apr 13, 2026
Merged

refactor(react): restructure sources list and sources-add container#220
RhysSullivan merged 1 commit into
rs/source-forms-04-other-formsfrom
rs/source-forms-05-sources-page

Conversation

@RhysSullivan

Copy link
Copy Markdown
Collaborator

5 of 5 — source-forms refactor stack (graphite copy of #191 by @mrzmyr). Stacks on #219.

  • pages/sources.tsx: restructures the list to use <CardStack> entries with title cards, moves connected sources to the top, and drops the old built-in section header (built-ins live exclusively in the sidebar now).
  • pages/sources-add.tsx: wraps the add-source container in a relative positioning wrapper so <FloatActions> can render as a sticky bottom bar inside the form without escaping the layout.

See #191 for full discussion.

Stack

  1. feat(react): add UI primitives for source forms #216 feat(react): add UI primitives for source forms
  2. feat(react): add <AuthenticationSection> primitive #217 feat(react): add <AuthenticationSection> primitive
  3. refactor(mcp): adopt shared primitives in Add/Edit source forms #218 refactor(mcp): adopt shared primitives in Add/Edit source forms
  4. refactor(sources): standardize openapi, graphql, google-discovery, onepassword forms #219 refactor(sources): standardize openapi, graphql, google-discovery, onepassword forms
  5. (this PR) refactor(react): restructure sources list and sources-add container

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Apr 13, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
executor-marketing 941612c Commit Preview URL

Branch Preview URL
Apr 13 2026, 07:36 AM

@RhysSullivan
RhysSullivan marked this pull request as ready for review April 13, 2026 05:49
@RhysSullivan
RhysSullivan force-pushed the rs/source-forms-04-other-forms branch from f87848f to f4981ad Compare April 13, 2026 05:51
@RhysSullivan
RhysSullivan force-pushed the rs/source-forms-05-sources-page branch from 2618ece to ab595f9 Compare April 13, 2026 05:51
@pkg-pr-new

pkg-pr-new Bot commented Apr 13, 2026

Copy link
Copy Markdown

Open in StackBlitz

@executor/sdk

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/sdk@220

@executor/plugin-file-secrets

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-file-secrets@220

@executor/plugin-google-discovery

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-google-discovery@220

@executor/plugin-graphql

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-graphql@220

@executor/plugin-keychain

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-keychain@220

@executor/plugin-mcp

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-mcp@220

@executor/plugin-onepassword

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-onepassword@220

@executor/plugin-openapi

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-openapi@220

@executor/plugin-workos-vault

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-workos-vault@220

commit: 247476c

@RhysSullivan
RhysSullivan force-pushed the rs/source-forms-04-other-forms branch from f4981ad to 86c62c1 Compare April 13, 2026 06:05
@RhysSullivan
RhysSullivan force-pushed the rs/source-forms-05-sources-page branch from ab595f9 to 3a295d5 Compare April 13, 2026 06:05
@RhysSullivan
RhysSullivan force-pushed the rs/source-forms-04-other-forms branch from 86c62c1 to c88f46c Compare April 13, 2026 06:07
@RhysSullivan
RhysSullivan force-pushed the rs/source-forms-05-sources-page branch from 3a295d5 to 33a2284 Compare April 13, 2026 06:07
@RhysSullivan
RhysSullivan force-pushed the rs/source-forms-04-other-forms branch from c88f46c to a203e02 Compare April 13, 2026 06:21
@RhysSullivan
RhysSullivan force-pushed the rs/source-forms-05-sources-page branch from 33a2284 to 7a947ed Compare April 13, 2026 06:21
@RhysSullivan
RhysSullivan force-pushed the rs/source-forms-04-other-forms branch from a203e02 to 6bef298 Compare April 13, 2026 07:00
@RhysSullivan
RhysSullivan force-pushed the rs/source-forms-05-sources-page branch from 7a947ed to dbbecc6 Compare April 13, 2026 07:00
@RhysSullivan
RhysSullivan force-pushed the rs/source-forms-04-other-forms branch from 6bef298 to a8cd5e4 Compare April 13, 2026 07:05
@RhysSullivan
RhysSullivan force-pushed the rs/source-forms-05-sources-page branch from dbbecc6 to f97f067 Compare April 13, 2026 07:05
@RhysSullivan
RhysSullivan force-pushed the rs/source-forms-04-other-forms branch from a8cd5e4 to dfb818f Compare April 13, 2026 07:21
@RhysSullivan
RhysSullivan force-pushed the rs/source-forms-05-sources-page branch from f97f067 to 247476c Compare April 13, 2026 07:21
- pages/sources.tsx: restructures the list to use CardStack entries with
  title cards, moves connected sources to the top, and drops the old
  built-in section header (built-ins live exclusively in the sidebar now).
- pages/sources-add.tsx: wraps the add-source container in a relative
  positioning wrapper so FloatActions can render as a sticky bottom bar
  inside the form without escaping the layout.
@RhysSullivan
RhysSullivan force-pushed the rs/source-forms-04-other-forms branch from dfb818f to f598bfc Compare April 13, 2026 07:34
@RhysSullivan
RhysSullivan force-pushed the rs/source-forms-05-sources-page branch from 247476c to 941612c Compare April 13, 2026 07:34
@RhysSullivan
RhysSullivan merged commit 1d721c2 into rs/source-forms-04-other-forms Apr 13, 2026
4 of 6 checks passed
@RhysSullivan
RhysSullivan deleted the rs/source-forms-05-sources-page branch April 13, 2026 08:13
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