Skip to content

feat: containerize API and add database options - #56

Merged
procoder-ji merged 3 commits into
procoderji-com:mainfrom
Coding-Professional:feat/container-database-options
Nov 10, 2026
Merged

procoder-ji merged 3 commits into
procoderji-com:mainfrom
Coding-Professional:feat/container-database-options

Conversation

@Coding-Professional

@Coding-Professional Coding-Professional commented Nov 10, 2026 •

Copy link
Copy Markdown
Collaborator

Containerize the ASP.NET Core API with PostgreSQL, SQL Server, or MongoDB. Neon PostgreSQL is the selected hosted setup, with pooled runtime and direct migration connections. Includes persistent Compose stacks, an optional Vercel backend config, and contributor documentation.

Validated API contracts against SQLite and all three servers, Compose startup/restart persistence, locked restore, and image smoke. Neon Compose configuration and Npgsql connection settings also validate. Live Neon/Vercel deployment requires operator credentials and has not been performed.

Copilot AI balanced review requested due to automatic review settings November 10, 2026 12:27

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

Copilot AI balanced review requested due to automatic review settings November 10, 2026 12:29

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

Copilot AI balanced review requested due to automatic review settings November 10, 2026 12:50

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

@procoder-ji procoder-ji left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @Coding-Professional for the container setup and database options. Reviewed the current implementation and Neon follow-up; no blocking changes requested.

Checked account identity/isolation, transaction retries, SQL Server migrations and snapshots, Mongo progress generation/revision checks, durable Data Protection keys, Docker/Compose, and CI. Neon correctly uses Production with a pooled runtime connection and a direct migration connection. The focused relational/account tests pass (8/8), and the Neon Compose configuration validates.

Three CI checks are green at review time; browser/SEO is still running. Live Neon connectivity, Vercel deployment and Google consent remain the documented operator verification steps.

@procoder-ji
procoder-ji merged commit 7d20d37 into procoderji-com:main Nov 10, 2026
4 checks passed
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.

3 participants