Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix: typo in web README - #3115

Open
mervivian wants to merge 1 commit into
onlook-dev:mainfrom
mervivian:fix/typo-preload-readme
Open

fix: typo in web README#3115
mervivian wants to merge 1 commit into
onlook-dev:mainfrom
mervivian:fix/typo-preload-readme

Conversation

@mervivian

@mervivian mervivian commented Jun 2, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown

Summary

There's a double-n typo in apps/web/README.md: "communnicating" should be "communicating". Small but looks unprofessional in a contributor-facing doc.

How to reproduce

Open apps/web/README.md and look at the description of the Preload section on line 10.

Test plan

Verify the word reads "communicating" in the updated file.

Summary by CodeRabbit

  • Documentation
    • Corrected a spelling error in the README's Preload section.

@vercel

vercel Bot commented Jun 2, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs-onlook Skipped Skipped Jun 2, 2026 1:22am

Request Review

@vercel
vercel Bot temporarily deployed to Preview – docs-onlook June 2, 2026 01:22 Inactive
@vercel

vercel Bot commented Jun 2, 2026

Copy link
Copy Markdown

@mervivian is attempting to deploy a commit to the Onlook Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Jun 2, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6eaa0801-e133-43dc-a8e1-636892d0a7f4

📥 Commits

Reviewing files that changed from the base of the PR and between a242be5 and 5471951.

📒 Files selected for processing (1)
  • apps/web/README.md
📝 Walkthrough

Walkthrough

This PR corrects a single typo in the Preload section of apps/web/README.md, fixing the spelling of "communicating" in the description of iframe DOM communication.

Changes

Documentation Typo Fix

Layer / File(s) Summary
Preload section spelling correction
apps/web/README.md		 Corrects the misspelling of "communicating" in the Preload section describing iframe DOM communication.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

Poem

A curious typo, now caught and corrected,
The Preload section's text, now perfectly reflected,
One letter fixed, communication flows clear,
Through iframe and DOM, the message rings true here! 🐰✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description deviates from the template structure; it lacks clear section headings like 'Description', 'Related Issues', 'Type of Change', 'Testing', and 'Screenshots'. Restructure the description to follow the provided template with proper section headings (Description, Related Issues, Type of Change, Testing, Additional Notes) for consistency.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix: typo in web README' directly and clearly summarizes the main change: correcting a typo in the web README documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@mervivian