Skip to content

updated production readiness page - #183

Merged
arhimede merged 3 commits into
mainfrom
issue-181
Oct 15, 2026
Merged

arhimede merged 3 commits into
mainfrom
issue-181

Conversation

@bidi47

@bidi47 bidi47 commented Oct 14, 2026

Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: bidi47 <bidi@apidemia.com>
@bidi47 bidi47 linked an issue Oct 14, 2026 that may be closed by this pull request
@bidi47
bidi47 requested a review from arhimede October 14, 2026 19:14
## Best practices

Two nginx settings worth making before you go live.
Both go in the `server` block that serves the API.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not sure that developers will know where to find this server block.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

i added a more detailed explanation of the file, its potential content and where the edit would go

Signed-off-by: bidi47 <bidi@apidemia.com>
@bidi47
bidi47 requested a review from alexmerlin October 15, 2026 09:32
## Best practices

Two nginx settings worth making before you go live.
Dotkernel API ships no nginx configuration, only the Apache `.htaccess` in `public/`, so both go in the nginx virtual host you write for the API.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

add a comment with what code to add in htaccess file to acheive the same goal

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

i added a new section to include both code and explanations

Signed-off-by: bidi47 <bidi@apidemia.com>
@bidi47
bidi47 requested a review from arhimede October 15, 2026 10:20
@arhimede
arhimede merged commit 9a9c0d8 into main Oct 15, 2026
10 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.

disable HTTP to HTTPS redirect

3 participants