docs: add Spanish README - #150
Conversation
📝 WalkthroughWalkthroughThe pull request adds ChangesSpanish README
Estimated code review effort: 1 (Trivial) | ~5 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@README.es-ES.md`:
- Around line 145-151: Update the createApolloClient example to resolve the
undefined GRAPHQL_ENDPOINT reference by reusing the existing GRAPHQL_URL
constant or declaring GRAPHQL_ENDPOINT before the HttpLink configuration.
- Around line 56-72: Update the ProfilePage example so its identifiers are
defined and consistent: import the project’s gql helper alongside the existing
imports, and pass the declared QUERY constant to useQuery instead of the
undefined PROFILE_QUERY.
- Around line 98-100: Update the Markdown code fence surrounding the
withApollo(ProfilePage, { ssr: false }) example in README.es-ES.md to specify
the JavaScript language using a js fence, while preserving the example content.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: a657ec48-d427-48fd-860b-a20409f3fb91
📒 Files selected for processing (1)
README.es-ES.md
| ```js | ||
| import React from 'react'; | ||
| import { useQuery } from '@apollo/react-hooks'; | ||
|
|
||
| import withApollo from 'hocs/withApollo'; | ||
|
|
||
| const QUERY = gql` | ||
| query Profile { | ||
| profile { | ||
| name | ||
| displayname | ||
| } | ||
| } | ||
| `; | ||
|
|
||
| const ProfilePage = () => { | ||
| const { loading, error, data } = useQuery(PROFILE_QUERY); |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟠 Major | ⚡ Quick win
Make the basic query example executable.
The snippet uses gql without importing it. It declares QUERY but passes PROFILE_QUERY to useQuery. A reader copying this example will get undefined identifiers. Add the project’s gql import and use QUERY, or rename the declaration.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@README.es-ES.md` around lines 56 - 72, Update the ProfilePage example so its
identifiers are defined and consistent: import the project’s gql helper
alongside the existing imports, and pass the declared QUERY constant to useQuery
instead of the undefined PROFILE_QUERY.
| ``` | ||
| export default withApollo(ProfilePage, { ssr: false }); | ||
| ``` |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Specify the code fence language.
Change the fence at Line 98 to ```js to satisfy Markdown linting and preserve syntax highlighting.
🧰 Tools
🪛 markdownlint-cli2 (0.23.1)
[warning] 98-98: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@README.es-ES.md` around lines 98 - 100, Update the Markdown code fence
surrounding the withApollo(ProfilePage, { ssr: false }) example in
README.es-ES.md to specify the JavaScript language using a js fence, while
preserving the example content.
Source: Linters/SAST tools
| const createApolloClient = ({ initialState, headers = {} }) => { | ||
| const authLink = () => setContext(attachAuth(headers)); | ||
|
|
||
| const httpLink = new HttpLink({ | ||
| credentials: 'include', | ||
| uri: GRAPHQL_ENDPOINT, | ||
| fetch |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟠 Major | ⚡ Quick win
Define the authenticated GraphQL endpoint.
The example references GRAPHQL_ENDPOINT at Line 150, but does not declare it. The earlier example declares a different constant, GRAPHQL_URL. Define GRAPHQL_ENDPOINT in this example or use the existing endpoint constant.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@README.es-ES.md` around lines 145 - 151, Update the createApolloClient
example to resolve the undefined GRAPHQL_ENDPOINT reference by reusing the
existing GRAPHQL_URL constant or declaring GRAPHQL_ENDPOINT before the HttpLink
configuration.
1 participant
Adds README.es-ES.md alongside the original documentation.
Summary by CodeRabbit