From a2acb70f69228dbb2af2556189a0cb5689247793 Mon Sep 17 00:00:00 2001 From: Joel Dickson Date: Thu, 26 Mar 2026 07:46:51 +0700 Subject: [PATCH 1/7] Docs: remove project structure section from README --- README.md | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/README.md b/README.md index 10617a3..7d35660 100644 --- a/README.md +++ b/README.md @@ -14,18 +14,6 @@ This is an internal tool designed to be deployed within your organization's infr - **Frontend:** React 19, TypeScript, Vite 8, Tailwind CSS 3, Tremor, Recharts - **Container:** Docker image available at `agoda/devex-telemetry` -## Project Structure - -```text -src/ -├── Agoda.DevExTelemetry.WebApi/ # ASP.NET Core API (controllers, Program.cs) -├── Agoda.DevExTelemetry.Core/ # Domain layer (entities, DTOs, services, DbContext) -├── Agoda.DevExTelemetry.IntegrationTests/ -├── Agoda.DevExTelemetry.UnitTests/ -├── Clientside/ # React SPA (Vite + Tailwind + Tremor) -└── Agoda.DevExTelemetry.sln -``` - ## Telemetry Clients These are the client libraries that instrument developer tooling and send telemetry to this server: From 9f1bac82ac4ff3f0cc99aebb868902006144398c Mon Sep 17 00:00:00 2001 From: Joel Dickson Date: Thu, 26 Mar 2026 07:48:26 +0700 Subject: [PATCH 2/7] Docs: remove ingest endpoints and PostgreSQL MVP migration notes --- README.md | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/README.md b/README.md index 7d35660..54cdea2 100644 --- a/README.md +++ b/README.md @@ -33,21 +33,6 @@ These are the client libraries that instrument developer tooling and send teleme | Jest reporter | npm | `POST /jest` | [testresults-collector](https://github.com/agoda-com/testresults-collector) | | Vitest reporter | npm | `POST /vitest` | [testresults-collector](https://github.com/agoda-com/testresults-collector) | -## Ingest Endpoints - -| Endpoint | Payload Type | -|---|---| -| `POST /dotnet` | .NET MSBuild compile, ASP.NET startup, first response | -| `POST /dotnet/nunit` | NUnit / xUnit test results | -| `POST /junit` | JUnit test results | -| `POST /jest` | Jest test results | -| `POST /vitest` | Vitest test results | -| `POST /scala/scalatest` | ScalaTest results (gzip supported) | -| `POST /webpack` | Webpack build metrics | -| `POST /vite` | Vite build / HMR metrics | -| `POST /gradletalaiot` | Gradle Talaiot build metrics | -| `POST /testdata/junit` | JUnit XML multipart upload | - ## Dashboard Pages - **Test Run Performance** — pass rates, durations, per-test-case drill-down @@ -64,8 +49,6 @@ export POSTGRES_CONNECTION_STRING='Host=localhost;Port=5432;Database=devex_telem Notes: - If `POSTGRES_CONNECTION_STRING` is not set, the app uses SQLite. -- Current PostgreSQL initialization uses `EnsureCreated()` as an MVP path. -- For long-term schema evolution, move to dedicated PostgreSQL migrations and `db.Database.Migrate()`. ## Docker Usage From 13d058b8408a4c4545733db334067a75fc730092 Mon Sep 17 00:00:00 2001 From: Joel Dickson Date: Thu, 26 Mar 2026 07:49:01 +0700 Subject: [PATCH 3/7] Docs: remove dashboard pages section from README --- README.md | 6 ------ 1 file changed, 6 deletions(-) diff --git a/README.md b/README.md index 54cdea2..073aa2c 100644 --- a/README.md +++ b/README.md @@ -33,12 +33,6 @@ These are the client libraries that instrument developer tooling and send teleme | Jest reporter | npm | `POST /jest` | [testresults-collector](https://github.com/agoda-com/testresults-collector) | | Vitest reporter | npm | `POST /vitest` | [testresults-collector](https://github.com/agoda-com/testresults-collector) | -## Dashboard Pages - -- **Test Run Performance** — pass rates, durations, per-test-case drill-down -- **API Build Performance** — compile, startup, and first response times -- **Clientside Build Performance** — hot reload vs full build metrics - ## PostgreSQL Support The API supports PostgreSQL when `POSTGRES_CONNECTION_STRING` is provided. From 81850bddfac06b46eb1a2d6a2f340bac59f1b970 Mon Sep 17 00:00:00 2001 From: Joel Dickson Date: Thu, 26 Mar 2026 07:51:19 +0700 Subject: [PATCH 4/7] Docs: expand local Vite proxy explanation and add blog link --- README.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/README.md b/README.md index 073aa2c..643bc90 100644 --- a/README.md +++ b/README.md @@ -116,6 +116,8 @@ npm run dev The dev server starts at `http://localhost:5173` and proxies `/api` requests to the backend. +For local development, this means the React/Vite app can call API routes without CORS pain or hard-coded environment switching—frontend and backend feel like one system while still running as separate processes. It keeps the feedback loop fast (UI hot reload + live API changes) and mirrors production routing behavior more closely than static mocks. I wrote more about this pattern on [Beer and Servers Don’t Mix](https://medium.com/beer-and-servers-dont-mix). + ### Running Tests ```bash From 1e99d86f091a4ff3a2ff54e84235a7a3b9e9d3d2 Mon Sep 17 00:00:00 2001 From: Joel Dickson Date: Thu, 26 Mar 2026 07:53:43 +0700 Subject: [PATCH 5/7] Docs: remove CI/CD section and link to specific Vite + .NET BFF post --- README.md | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/README.md b/README.md index 643bc90..84ca7ac 100644 --- a/README.md +++ b/README.md @@ -116,7 +116,7 @@ npm run dev The dev server starts at `http://localhost:5173` and proxies `/api` requests to the backend. -For local development, this means the React/Vite app can call API routes without CORS pain or hard-coded environment switching—frontend and backend feel like one system while still running as separate processes. It keeps the feedback loop fast (UI hot reload + live API changes) and mirrors production routing behavior more closely than static mocks. I wrote more about this pattern on [Beer and Servers Don’t Mix](https://medium.com/beer-and-servers-dont-mix). +For local development, this means the React/Vite app can call API routes without CORS pain or hard-coded environment switching—frontend and backend feel like one system while still running as separate processes. It keeps the feedback loop fast (UI hot reload + live API changes) and mirrors production routing behavior more closely than static mocks. I wrote more about this exact setup in [Bridging Worlds: Making .NET BFF and React/Vite Play Nice in Development](https://medium.com/beer-and-servers-dont-mix/bridging-worlds-making-net-bff-and-react-vite-play-nice-in-development-b0f7b9311790). ### Running Tests @@ -124,12 +124,3 @@ For local development, this means the React/Vite app can call API routes without cd src dotnet test ``` - -## CI/CD - -Pushes to `main` trigger automatic deployment to Azure App Service via GitHub Actions. - -| Workflow | Trigger | Purpose | -|---|---|---| -| `build.yml` | Pull requests to main | Build + test validation | -| `deploy.yml` | Push to main | Build + deploy to Azure | From e3d2c27a7f115ac3fe6c5ef34e9425297fdc1021 Mon Sep 17 00:00:00 2001 From: Joel Dickson Date: Thu, 26 Mar 2026 07:59:43 +0700 Subject: [PATCH 6/7] Docs: remove extended local-development proxy paragraph --- README.md | 2 -- 1 file changed, 2 deletions(-) diff --git a/README.md b/README.md index 84ca7ac..ec45378 100644 --- a/README.md +++ b/README.md @@ -116,8 +116,6 @@ npm run dev The dev server starts at `http://localhost:5173` and proxies `/api` requests to the backend. -For local development, this means the React/Vite app can call API routes without CORS pain or hard-coded environment switching—frontend and backend feel like one system while still running as separate processes. It keeps the feedback loop fast (UI hot reload + live API changes) and mirrors production routing behavior more closely than static mocks. I wrote more about this exact setup in [Bridging Worlds: Making .NET BFF and React/Vite Play Nice in Development](https://medium.com/beer-and-servers-dont-mix/bridging-worlds-making-net-bff-and-react-vite-play-nice-in-development-b0f7b9311790). - ### Running Tests ```bash From 0623a779aa0f158747888e8f9dca745db74c7666 Mon Sep 17 00:00:00 2001 From: Joel Dickson Date: Thu, 26 Mar 2026 08:01:05 +0700 Subject: [PATCH 7/7] Docs: keep specific Vite + .NET BFF blog link sentence --- README.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/README.md b/README.md index ec45378..d772661 100644 --- a/README.md +++ b/README.md @@ -116,6 +116,8 @@ npm run dev The dev server starts at `http://localhost:5173` and proxies `/api` requests to the backend. +I wrote more about this exact setup in [Bridging Worlds: Making .NET BFF and React/Vite Play Nice in Development](https://medium.com/beer-and-servers-dont-mix/bridging-worlds-making-net-bff-and-react-vite-play-nice-in-development-b0f7b9311790). + ### Running Tests ```bash