Skip to content

Conversation

@dhtclk
Copy link
Collaborator

@dhtclk dhtclk commented Oct 16, 2025

Summary

nginx set up guide for clickstack

Checklist

@vercel
Copy link

vercel bot commented Oct 16, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
clickhouse-docs Ready Ready Preview Comment Oct 29, 2025 5:44pm
3 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
clickhouse-docs-jp Ignored Ignored Oct 29, 2025 5:44pm
clickhouse-docs-ru Ignored Ignored Preview Oct 29, 2025 5:44pm
clickhouse-docs-zh Ignored Ignored Preview Oct 29, 2025 5:44pm

@dhtclk dhtclk marked this pull request as ready for review October 28, 2025 17:29
@dhtclk dhtclk requested a review from a team as a code owner October 28, 2025 17:29

| Section | Description |
|------|-------------|
| [Nginx Logs](./nginx-logs.md) | Introduction to data ingestion methods and architecture |
Copy link
Collaborator

Choose a reason for hiding this comment

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

I thought we prefer path links not .md files

@dhtclk dhtclk merged commit 8be8ec2 into main Oct 29, 2025
15 checks passed
@Blargian Blargian mentioned this pull request Oct 30, 2025
3 tasks
@Blargian
Copy link
Member

Blargian commented Oct 30, 2025

@dhtclk I don't mind if you override the merge check but please be more careful. You did not address all the feedback and have turned off the anchor checker.

#4668 to fix

We also discussed placing files in the examples repo rather than /static

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.

4 participants