docs: remove duplicate console.info from beforeLoad example

[szhsin]

Updated the example to call fetchPosts() directly instead of wrapping it in console.info(), reflecting the intended usage.

Summary by CodeRabbit

• Documentation
  • Updated the data-loading guide example to remove console logging from the route loader pattern, providing a cleaner approach to data-fetching operations.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

A documentation example in the data-loading guide was modified to remove a logging side effect. The fetchPosts function invocation remains, but the console.info call that previously printed its result to the console was removed from the route loader example.

Changes

Cohort / File(s)	Summary
Documentation example update docs/router/framework/react/guide/data-loading.md	Removed console.info logging from route loader example; fetchPosts call still executes but no longer outputs result to console

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

This is a straightforward documentation example edit with no functional code changes, logic updates, or API modifications.

Poem

🐰 A logger hushed, a cleaner flow,
The fetch still works—just silent now,
No console chatter, just the show,
Of data loaded, anyhow! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: removing a duplicate console.info call from a beforeLoad example in the documentation.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 41dba10 and 9c82159.

📒 Files selected for processing (1)

• docs/router/framework/react/guide/data-loading.md (1 hunks)

🧰 Additional context used

📓 Path-based instructions (1)

docs/**/*.{md,mdx}

📄 CodeRabbit inference engine (AGENTS.md)

Use relative links to docs/ folder format (e.g., ./guide/data-loading) for internal documentation references

Files:

• docs/router/framework/react/guide/data-loading.md

🔇 Additional comments (1)

docs/router/framework/react/guide/data-loading.md (1)

314-329: Good catch on removing the redundant logging.

The change clarifies the example by removing the outer console.info() wrapper around fetchPosts(), since the injected fetchPosts function (line 321) already calls console.info() internally. This makes the intended usage pattern clearer without the duplicate logging side effect.

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}