Skip to content

docs: clarify DB env overrides#146

Open
dex-the-ai wants to merge 3 commits into
mainfrom
dex/aspnet-readme-env-overrides-2026-07-07
Open

docs: clarify DB env overrides#146
dex-the-ai wants to merge 3 commits into
mainfrom
dex/aspnet-readme-env-overrides-2026-07-07

Conversation

@dex-the-ai

@dex-the-ai dex-the-ai commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • update the README setup instructions to mention the supported DB_CONN_STR, DB_USERNAME, and DB_PASSWORD environment-variable overrides
  • keep this sweep change intentionally small and doc-focused because the larger dependency refresh is already covered by maintainer PR chore(deps): update .NET dependencies #145

Verification

  • docker run --rm -v /home/elliot/.openclaw/state/tutorial-maintenance/repos/aspnet-quickstart:/repo -w /repo mcr.microsoft.com/dotnet/sdk:8.0 bash -lc "dotnet restore Org.Quickstart.sln && dotnet build Org.Quickstart.sln --configuration Debug --no-restore" → build passed
  • docker run --rm -e DB_CONN_STR -e DB_USERNAME -e DB_PASSWORD -v /home/elliot/.openclaw/state/tutorial-maintenance/repos/aspnet-quickstart:/repo -w /repo mcr.microsoft.com/dotnet/sdk:8.0 bash -lc "dotnet restore Org.Quickstart.sln && dotnet test src/Org.Quickstart.IntegrationTests/Org.Quickstart.IntegrationTests.csproj --configuration Debug --verbosity normal" → environment-blocked by missing travel-sample bucket on the configured Couchbase target
  • containerized app run at http://127.0.0.1:18080 rendered Swagger successfully:
    • GET /swagger/v1/swagger.json200 OK, OpenAPI 3.0.4, 7 paths
    • GET /index.html → Swagger UI rendered successfully
    • GET /api/v1/airport/list?country=France&limit=1500 InternalServerError with Ensure that you have the travel-sample bucket loaded in the cluster.
  • detailed notes: tutorial-maintenance/runs/aspnet-quickstart/2026-07-07T094906-0700/verification.md

Evidence

  • README now matches the actual runtime behavior used by local/dev container workflows: DB settings can come from appsettings.Development.json or the DB_* environment variables.
  • Visual proof from the running Swagger surface:
    • tutorial-maintenance/runs/aspnet-quickstart/2026-07-07T094906-0700/screenshots/swagger-ui.png
    • tutorial-maintenance/runs/aspnet-quickstart/2026-07-07T094906-0700/video/swagger-ui.webm
  • Request notes captured in tutorial-maintenance/runs/aspnet-quickstart/2026-07-07T094906-0700/walkthrough.txt

Notes

  • The host only has the .NET 10 runtime installed, so the verification for this run used Dockerized .NET 8 instead of native dotnet test.
  • I did not duplicate the dependency refresh already open in PR chore(deps): update .NET dependencies #145; this branch is just the honest README drift fix surfaced during the sweep.

Media evidence

  • Swagger UI screenshot and short walkthrough video captured from the running app on this branch.

Swagger Ui

Notes
  • Local artifact path: tutorial-maintenance/runs/aspnet-quickstart/2026-07-07T094906-0700/screenshots/swagger-ui.png

Swagger Ui

Notes
  • Local artifact path: tutorial-maintenance/runs/aspnet-quickstart/2026-07-07T094906-0700/video/swagger-ui.webm

@dex-the-ai

Copy link
Copy Markdown
Contributor Author

Owner ping for this maintenance PR: @azaddhirajkumar

Requesting review from the manifest-listed owner(s) where GitHub does not already show a pending review request.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates the README.md to document the support for overriding default Couchbase configuration values using the DB_CONN_STR, DB_USERNAME, and DB_PASSWORD environment variables. The review feedback correctly points out that all three environment variables must be set simultaneously for the override to take effect, and suggests updating the documentation to reflect this requirement accurately.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread README.md Outdated
Comment thread README.md Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants