Skip to content

fix: correct link to testing library docs#154

Merged
Huxpro merged 1 commit into
lynx-family:mainfrom
upupming:feat/testing-library-link
May 9, 2025
Merged

fix: correct link to testing library docs#154
Huxpro merged 1 commit into
lynx-family:mainfrom
upupming:feat/testing-library-link

Conversation

@upupming
Copy link
Copy Markdown
Collaborator

@upupming upupming commented May 9, 2025

Fix https://lynxjs.org/react/react-lynx-testing-library.html -> https://lynxjs.org/react/reactlynx-testing-library.html

Change-Id: I76fff2fce65d39cdb919b68e208015f6629a0c9e
@upupming upupming requested a review from Huxpro May 9, 2025 12:50
colinaaa pushed a commit to lynx-family/lynx-stack that referenced this pull request May 9, 2025
<!--
  Thank you for submitting a pull request!

We appreciate the time and effort you have invested in making these
changes. Please ensure that you provide enough information to allow
others to review your pull request.

Upon submission, your pull request will be automatically assigned with
reviewers.

If you want to learn more about contributing to this project, please
visit:
https://github.com/lynx-family/lynx-stack/blob/main/CONTRIBUTING.md.
-->

## Summary

<!-- Can you explain the reasoning behind implementing this change? What
problem or issue does this pull request resolve? -->

<!-- It would be helpful if you could provide any relevant context, such
as GitHub issues or related discussions. -->

Fix `https://lynxjs.org/react/react-lynx-testing-library.html` ->
`https://lynxjs.org/react/reactlynx-testing-library.html`

See also: lynx-family/lynx-website#154

## Checklist

<!--- Check and mark with an "x" -->

- [ ] Tests updated (or **not required**).
- [ ] Documentation updated (or **not required**).
Copy link
Copy Markdown
Collaborator

@colinaaa colinaaa left a comment

Choose a reason for hiding this comment

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

I'm not sure why this isn't addressed by the dead link check :)

Copy link
Copy Markdown
Collaborator

@colinaaa colinaaa left a comment

Choose a reason for hiding this comment

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

Oh I see. We should not use http link.

Comment thread docs/en/api/lynx-testing-environment/index.mdx
Comment thread docs/zh/api/lynx-testing-environment/index.mdx
Comment thread docs/zh/api/lynx-testing-environment/index.mdx
Copy link
Copy Markdown
Collaborator

@Huxpro Huxpro left a comment

Choose a reason for hiding this comment

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

Since it's type generated I think using absolute URL is good

@Huxpro Huxpro merged commit 5589fa9 into lynx-family:main May 9, 2025
1 check 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