Switch `connectors` over to using GQL by travjenkins · Pull Request #1949 · estuary/ui

travjenkins · 2026-04-03T14:32:22Z

Issues

#1929

Changes

issue_number

We now fetch all of the connector details from GQL on load of edit/create and the admin section (test page still uses PostgREST). This means we stop making extra calls for one offs like resource schema. This is the new useConnectorTag context.
More forceful with checking that connectors are valid. The GQL endpoint requires a lot more info so the UI should be good to trust the responses more. This impacts things like the "valid connector" wrapper and the hasValidConnectors hook we used to need.
test/jsonforms is now available in production
No longer tracking unsupportedConnector as we just fired an event for that and didn't show anything to the user.
Cleaned up the old details hydrator as only the test page was using that

Tests

Manually tested

scenarios you manually tested

Automated tests

unit testing covered

Playwright tests ran locally

Screenshots

Edit now shows the connector image tag in the URL

…lected and the user is not allowed to change it later

…ctors_v2

Handling errors when fetching Cleaning up the unsupported stuff

new context blocks showing the entire flow if there isn't a connector

Adding in fetching 500 so we don't need pagination Updating typing now that my local has latest merged on flow

…that has been handled

it a pretty safe page to allow hitting.

…ctors_v2

travjenkins · 2026-04-10T15:34:54Z

src/components/capture/Create/index.tsx

-        setDraftId(null);
-        setInitiateDiscovery(true);
-    }, [setDraftId, setInitiateDiscovery, imageTag]);
-


Since we prevented users from changing the connector awhile ago I think this is totally safe to remove.

travjenkins · 2026-04-10T15:37:49Z

src/components/connectors/Grid/ConnectorCards.tsx

+        const q = searchQuery.toLowerCase();
+        return nodes.filter(
+            (node) =>
+                node.title?.toLowerCase().includes(q) ||
+                node.shortDescription?.toLowerCase().includes(q)
+        );


We now need to do some of the filtering client side.

travjenkins · 2026-04-10T15:44:21Z

src/components/shared/Entity/DetailsForm/useFormFields.ts

We used to need to set stuff here but seems it is good now.

Only thing to call out is the setConnector used to have a side effect of setting setEntityNameChanged. I think Kiahna is right and this was back when the connector could change and we needed to set the name changed because the connector being suffixed was going to change.

Adding basic eventing in Handling exceptions

travjenkins · 2026-04-10T16:52:08Z

src/context/Router/index.tsx

Connector folks could use the jsonforms test page in prod.

travjenkins · 2026-04-10T17:01:41Z

src/components/shared/guards/SupportRole.tsx

New basic guard for support role. Potentially we'l add a special error or something here but not major concern right now.

travjenkins · 2026-04-10T17:13:56Z

src/validation/index.ts

 );

-export const MAC_ADDR_RE = new RegExp(/^([0-9A-F]{2}:){7}([0-9A-F]{2})$/i);
+export const MAC_ADDR_LIKE_RE = new RegExp(/^([0-9A-F]{2}){7}([0-9A-F]{2})$/i);


GQL returns UUIDs without the colon vs Postgrest that returns the UUIDs with colons.

travjenkins added 12 commits March 31, 2026 15:36

Initial switching over to use GQL for connectors on the card list page

7e09fcc

Switching over to storing the connector stuff in context

e6b270f

Formatting to GQL format of UUIDs

f2de20c

Starting to clean up when we allowed connector to be changed

d10afa2

We do not need to enable setting this as we force the option to be se…

6b957cf

…lected and the user is not allowed to change it later

formatting stuff

c86d33a

Wiring back up the check for unsupported connectors

e63dc70

Merge remote-tracking branch 'origin/main' into travjenkins/gql/conne…

c820b09

…ctors_v2

Cleaning up old code/approach

4550725

Handling errors when fetching Cleaning up the unsupported stuff

fetching binding stuff from new context

92c4708

cleaning up the valid connector check - the backend checks this and the

e4003be

new context blocks showing the entire flow if there isn't a connector

cleaning up some typing

58a9118

travjenkins mentioned this pull request Apr 6, 2026

Switch connectors to gql #1931

Closed

6 tasks

travjenkins added 12 commits April 6, 2026 16:11

Cleaning up stuff we should not need anymore

7a152b2

Adding in fetching 500 so we don't need pagination Updating typing now that my local has latest merged on flow

Removing the old details hydrator for details as only test page used it

8b8f4dc

Cleaning up a lot of code just for the test json forms page

3621d53

Adding a bit more error handling

a93da55

Updating query so we fetch the image tag if available

b00508f

I think we are good to remove this - gonna test it out

326faa2

We can remove this as it was only around for test/jsonforms page and …

b21977f

…that has been handled

jsonforms test page will help connector team with quicker testing and it

3aabcde

it a pretty safe page to allow hitting.

Cleaning up some old code to move towards using the new context

abc5ea2

Some final cleanup of hanging code

f03eca2

Merge remote-tracking branch 'origin/main' into travjenkins/gql/conne…

1244c83

…ctors_v2

Post merge codegen run

679d56b

travjenkins commented Apr 10, 2026

View reviewed changes

travjenkins added 2 commits April 10, 2026 11:49

Cleaning up some old stuff

2bc46b3

Adding basic eventing in Handling exceptions

More cleanup

c2d3084

Hook no longer needed

1314862

travjenkins commented Apr 10, 2026

View reviewed changes

src/context/Router/index.tsx

Copy link
Copy Markdown

Member Author

travjenkins Apr 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Connector folks could use the jsonforms test page in prod.

travjenkins added 2 commits April 10, 2026 12:56

No clue why I thought we could remove this

44a1427

Adding support check wrapper around json test forms

90bbaa6

travjenkins commented Apr 10, 2026

View reviewed changes

travjenkins added 2 commits April 10, 2026 13:04

Hook not needed anymore

ee2a6c5

This is not needed for the test pages

13f2697

travjenkins commented Apr 10, 2026

View reviewed changes

some formatting issues and ignoring storybook

23997df

travjenkins added the change:planned This is a planned change label Apr 10, 2026

travjenkins marked this pull request as ready for review April 10, 2026 18:03

travjenkins requested a review from a team as a code owner April 10, 2026 18:03

travjenkins changed the title ~~Travjenkins/gql/connectors v2~~ Switch connectors over to using GQL Apr 10, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Switch `connectors` over to using GQL#1949

Switch `connectors` over to using GQL#1949
travjenkins wants to merge 32 commits intomainfrom
travjenkins/gql/connectors_v2

travjenkins commented Apr 3, 2026 •

edited

Loading

Uh oh!

travjenkins Apr 10, 2026

Uh oh!

travjenkins Apr 10, 2026

Uh oh!

travjenkins Apr 10, 2026

Uh oh!

travjenkins Apr 10, 2026

Uh oh!

travjenkins Apr 10, 2026

Uh oh!

travjenkins Apr 10, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

travjenkins commented Apr 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Issues

Changes

issue_number

Tests

Manually tested

Automated tests

Playwright tests ran locally

Screenshots

Uh oh!

travjenkins Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

travjenkins Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

travjenkins Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

travjenkins Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

travjenkins Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

travjenkins Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

travjenkins commented Apr 3, 2026 •

edited

Loading