Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix typos #4972

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix typos #4972

wants to merge 2 commits into from

Conversation

PixelPil0t1
Copy link

  1. Changes in README.md
    Modified Sentence in Introduction
    Old: Space Admin can add this plugin into your space.
    New: A Space Admin can add this plugin to your space.
    Reason: The phrase "into your space" was replaced with "to your space" because "add to" is the correct collocation in English. Also, "A Space Admin" was added for grammatical correctness.
    Fixed Grammar in Notice Section
    Old: notice: The domain should not included in .
    New: notice: The domain should not be included in .
    Reason: The verb "included" was missing the auxiliary verb "be", making the sentence grammatically incorrect.
  2. Changes in index.ts
    Fixed Grammar in Comment
    Old: // If the fetch fails: the event doesn't exists for this poap yet
    New: // If the fetch fails: the event doesn't exist for this poap yet
    Reason:
    "doesn't exists" → "doesn't exist": The verb "exist" should be in its base form after "doesn't."

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.

1 participant