docs: mcp connectors images - #2980
Conversation
Reuses the announcement blog post screenshots, cropped to the modals. Requested by Jan in #product-docs.
TC-MO
added
documentation
|
🗑️ Preview for this PR was deleted. |
The Lint images check rejects raw PNGs; ran pnpm opt:images.
marcel-rbro
left a comment
marcel-rbro
left a comment
There was a problem hiding this comment.
Some changes are still needed. Considering this is a quick fix we don't need to have the screenshots perfect, but there are some inconsistencies.
| @@ -67,6 +69,8 @@ The **MCP connectors** section lets you authorize third-party MCP servers (such | |||
| - _OAuth_ - the server supports OAuth and Apify can either register an OAuth client automatically (Dynamic Client Registration) or use an Apify-managed OAuth client. A consent screen opens in a popup. Grant access and close the popup. | |||
| - _Own OAuth client_ - the server supports OAuth but you need to register your own OAuth app with the provider (see below). Enter your client ID, client secret, authorization URL, and token URL, then complete the OAuth consent flow. | |||
|
|
|||
|  | |||
|
|
|||
There was a problem hiding this comment.
The labels in the docs prose are slightly different from the ones on the screen
|  | ||
|
|
There was a problem hiding this comment.
The description and the prose say connector picker, but the field is actually called "External data sources". Might be confusing in theory, I'd mention the caption at least once.
There was a problem hiding this comment.
The password manager extension is visible here. Although, it's not a big deal if these screens will be remade soon.
There was a problem hiding this comment.
good catch it will be trivial to white it out so I'll do so
There was a problem hiding this comment.
Shouldn't these 2 images be in the "sources/platform/integrations/ai/mcp-connectors/images" folder instead of the top-level integrations images folder?
There was a problem hiding this comment.
Shouldn't these 2 images be in the "sources/platform/integrations/ai/mcp-connectors/images" folder instead of the top-level integrations images folder?
Also name the picker's example label and link the own-OAuth-client section instead of saying 'see below'.
3 participants
Add Console screenshots to the MCP connector pages.
Reuses the blog post screenshots, cropped.
Requested on slack.