Skip to content

Conversation

@ZmmBigdata
Copy link
Contributor

Purpose

Linked issue: close #xxx (请替换为你关联的 GitHub Issue 编号)

What is the purpose of the change:
The link in the document is 404
https://seatunnel.apache.org/zh-CN/docs/2.3.9/connector-v2/source/Http/
https://seatunnel.apache.org/docs/2.3.9/connector-v2/source/Http/
cefac24306a46f91c871b31c27439c2

Brief change log

Please describe the changes:

  1. Modified the link address in the document

@davidzollo davidzollo added the First-time contributor First-time contributor label Jul 10, 2025
Copy link
Member

@Carl-Zhou-CN Carl-Zhou-CN left a comment

Choose a reason for hiding this comment

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

LGTM,Thank you for your contribution

@corgy-w corgy-w merged commit e6b7215 into apache:dev Jul 11, 2025
6 checks passed
@davidzollo
Copy link
Contributor

Hi @ZmmBigdata
Thanks for your first contribution.
Official website repo https://github.com/apache/seatunnel-website will sync this docs dir to seatunnel-website through https://github.com/apache/seatunnel-website/blob/main/.github/workflows/deploy.yml and then generate the newest content at 05:00 every day, you can check https://seatunnel.apache.org/docs/connector-v2/source/Http tomorrow. I think this url should use https://github.com/apache/seatunnel/blob/dev/seatunnel-e2e/seatunnel-connector-v2-e2e/connector-http-e2e/src/test/resources/mockserver-config.json

chncaesar pushed a commit to chncaesar/seatunnel that referenced this pull request Jul 19, 2025
dybyte pushed a commit to dybyte/seatunnel that referenced this pull request Jul 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants