Skip to content

Simplify README quick start #152

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

cjayyy
Copy link
Contributor

@cjayyy cjayyy commented Jun 10, 2025

Summary

  • document quick MCP remote server setup and Cursor install link
  • collapse local MCP server instructions for advanced setups

https://chatgpt.com/codex/tasks/task_b_684863e5b5708333a0a7f3d4896bff07

@cjayyy cjayyy added the codex label Jun 10, 2025 — with ChatGPT Connector
@cjayyy cjayyy requested a review from jordanrburger June 10, 2025 18:17
@cjayyy cjayyy marked this pull request as ready for review June 10, 2025 18:48
@cjayyy cjayyy self-assigned this Jun 10, 2025
@cjayyy
Copy link
Contributor Author

cjayyy commented Jun 11, 2025

WDYT about this PR? @jordanrburger

@cjayyy cjayyy requested a review from davidesner June 11, 2025 00:25
{
"mcpServers": {
"keboola": {
"url": "https://mcp.us-east4.gcp.keboola.com/sse"
Copy link
Contributor

Choose a reason for hiding this comment

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

This doesn't work for me in Cursor.

I needed to pass it through mcp-remote via npx :(

    "keboola": {
      "command": "npx",
      "args": [
        "mcp-remote",
        "https://mcp.us-east4.gcp.keboola.com/sse"
      ]
    }

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Stepan is also having trouble with this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ℹ️ You both use Linux.

Copy link
Contributor

Choose a reason for hiding this comment

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

@cjayyy Please add a note in text that the "quick setup" link does not work on Linux.

> When you are already a Keboola user, make sure to change MCP url stack in configuration JSON correspondingly (see [Keboola regions](#keboola-region) section).

### Other ways how to connect
<details>
Copy link
Contributor

Choose a reason for hiding this comment

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

is using HTML tags in Markdown a standard thing?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's not standard Markdown but GitHub enhanced Markdown.
See the docs here.

{
"mcpServers": {
"keboola": {
"url": "https://mcp.us-east4.gcp.keboola.com/sse"
Copy link
Contributor

Choose a reason for hiding this comment

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

@cjayyy Please add a note in text that the "quick setup" link does not work on Linux.

```

> [!NOTE]
> When you are already a Keboola user, make sure to change MCP url stack in configuration JSON correspondingly (see [Keboola regions](#keboola-region) section).
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't we also explain what stack is, somewhere in this readme?


Click to install:

[![Install MCP Server](https://cursor.com/deeplink/mcp-install-dark.svg)](https://cursor.com/install-mcp?name=keboola&config=eyJ1cmwiOiJodHRwczovL21jcC51cy1lYXN0NC5nY3Aua2Vib29sYS5jb20vc3NlIn0%3D)
Copy link
Contributor

Choose a reason for hiding this comment

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

Should not we add other stacks too, either here or somewhere in the readme?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants