-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
NV-1618 - Add Missing APNS Docs #3018
Conversation
NV-1618 Missing APNS docs
We have APNs provider in Push Channel but its documentation page is missing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🌟
Just 2 typos to rollback. Great adding this missing doc!
docs/docs/channels/push/fcm.md
Outdated
```ts | ||
import { Novu } from '@novu/node'; | ||
import { Novu } from '@node'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
import { Novu } from '@node'; | |
import { Novu } from '@novu/node'; |
Please rollback the change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, will do
docs/docs/channels/push/fcm.md
Outdated
Before triggering the notification to a subscriber(user) with push as a step in the workflow, make sure you have added the subscriber's device token as follows: | ||
|
||
```ts | ||
import { Novu, PushProviderIdEnum } from '@novu/node'; | ||
import { Novu, PushProviderIdEnum } from '@node'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And here too. 🙂
import { Novu, PushProviderIdEnum } from '@node'; | |
import { Novu, PushProviderIdEnum } from '@novu/node'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for catching
'loc-key' : 'GAME_PLAY_REQUEST_FORMAT', | ||
'loc-args' : [ 'Shelly', 'Rick'] | ||
}, | ||
sound: 'demo.wav' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is demo.wav
an attachment or some static name?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a static name, it can be any thing
docs/docs/channels/push/apns.md
Outdated
|
||
const novu = new Novu(process.env.NOVU_API_KEY); | ||
|
||
const body = req.body; // From your HTTPS listener |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this line can be removed as body is not used in setCredentials
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh makes sense.
What change does this PR introduce?
This adds the docs to help activate and integrate apns. It also improves the firebase set up documentation.
Why was this change needed?
Docs is non-existent.
Other information (Screenshots)