Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

Two part reg #3

Closed
wants to merge 4 commits into from
Closed

Two part reg #3

wants to merge 4 commits into from

Conversation

secondsun
Copy link
Contributor

Motivation

This adds registering as a part separate from initialization.

What

Android and iOS modules were refactored such that the javascript "init" method setups up the Push library, and "register" will cause a push registration. Previously this was combined into a single step under the init method.

Why

The init object and a registration object are separate objects, and the operations under the hood are separate as well. This had only been a single action out of historical curiosity.

Verification Steps

Use the "rn" branch of unifiedpush-cookbook. You will also need to setup a FCM and APNS account and configure UPS accordingly. You will also need to setup the ios and Android applications for your own services or reach out to us and test ours.

  1. Send a push
  2. Confirm push messages are received on andriod and iOS

@secondsun secondsun requested a review from ziccardi May 11, 2020 20:03
ziccardi and others added 2 commits May 14, 2020 18:28
Added a custom AppDelegate class that can be extended by the end user to avoid the need of a lot of boilerplate code
feat: added UPSEnableAppDelegate
@secondsun secondsun closed this May 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants