Update page structure of server quickstart, additional small edits#33
Update page structure of server quickstart, additional small edits#33feorlen wants to merge 1 commit intoagentcontrol:mainfrom
Conversation
| "logic": "any", | ||
| "match_mode": "contains", | ||
| "case_sensitive": false | ||
| "case_sensitive": False |
There was a problem hiding this comment.
This example did not work for me because false was undefined. Both "false" and False worked, changed this to False to make it more obvious it is a bool.
|
|
||
| - **Node.js 18+** — for the local development web dashboard (optional) | ||
|
|
||
| This tutorial was tested on macOS. |
There was a problem hiding this comment.
I know this was tested on macOS, but not which versions. It would be good to add version detail if available.
Just sticking this line here isn't my favorite, but it's a clue if someone runs into trouble on their system. Thorough testing of OS requirements is beyond the scope of this PR.
For me, it worked on Ubuntu 24.04 but not 22.04.
| > 💡 **Verify the server:** Open [http://localhost:8000/health](http://localhost:8000/health) — you should see `{"status": "healthy", "version": "..."}`. | ||
|
|
||
| ## Step 2: Install the SDK | ||
| #### Install the SDK |
There was a problem hiding this comment.
H4s are less than ideal but work. Perhaps revisit with a larger overhaul in the future.
| ## Prerequisites | ||
|
|
||
| Before starting the UI, make sure the **Agent Control server** is already running. If you haven't set it up yet, follow the [Quickstart](/core/quickstart) first. | ||
| Before starting the UI, make sure the **Agent Control server** is already running. If you haven't set it up yet, follow the [Quickstart](/core/quickstart) first. This tutorial was tested on macOS. |
There was a problem hiding this comment.
| - **pnpm 9+** — install from [pnpm.io](https://pnpm.io/) or run `corepack enable && corepack prepare pnpm@latest --activate` | ||
|
|
||
| > **Note:** If you started the server using **One-line setup** from the [Quickstart](/core/quickstart), the UI is already running at [http://localhost:4000](http://localhost:4000). You can skip the setup steps below and jump directly to [viewing the dashboard](#create-controls-from-the-ui). | ||
| > **Note:** If you started the server using **One-line setup** from the [Quickstart](/core/quickstart), the UI is already running at [http://localhost:4000](http://localhost:4000). You can skip the setup steps below and jump directly to [Create Controls from the UI](#create-controls-from-the-ui) to add a control. |
There was a problem hiding this comment.
The original text for this link was not clear where it was sending you. Updated to match the target header.
1 participant
Several changes to clarify and improve the Quickstart and UI Quickstart pages.
quickstart.mdx:
uvto the list of prerequisites.ui-quickstart.mdx:
built-in-evaluators.mdx:
Shortcut ticket
For Galileo internally raised PRs only, please update this with your shortcut ticket.
SC-number
For external PRs, please add the issue (just put the number after the # below, and GitHub will automatically create a link):
No linked issue.
Checklist before requesting a review