Skip to content

Fix typo in README about serviceWorker subscription#116

Open
marcelokanzaki wants to merge 2 commits intozaru:masterfrom
marcelokanzaki:patch-1
Open

Fix typo in README about serviceWorker subscription#116
marcelokanzaki wants to merge 2 commits intozaru:masterfrom
marcelokanzaki:patch-1

Conversation

@marcelokanzaki
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown

@muhammadk1607 muhammadk1607 left a comment

Choose a reason for hiding this comment

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

The typo fix is itself a typo, :)

Comment thread README.md Outdated
1. Your server has (optionally) generated (one-time) a set of [Voluntary Application server Identification (VAPID)](https://tools.ietf.org/html/draft-ietf-webpush-vapid-01) keys. Otherwise, to send messages through Chrome, you have registered your site through the [Google Developer Console](https://console.developers.google.com/) and have obtained a GCM sender id and GCM API key from your app settings.
2. A `manifest.json` file, linked from your user's page, identifies your app settings.
3. Also in the user's web browser, a `serviceWorker` is installed and activated and its `pushManager` property is subscribed to push events with your VAPID public key, with creates a `subscription` JSON object on the client side.
3. Also in the user's web browser, a `serviceWorker` is installed and activated and its `pushManager` property is subscribed to push events with your VAPID public key, wich creates a `subscription` JSON object on the client side.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

wich -> which

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

This word is hard hahaha.

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.

2 participants