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

Remove onNewToken() event #137

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

Conversation

vfa-thanhnh
Copy link
Contributor

@vfa-thanhnh vfa-thanhnh commented Nov 19, 2018

概要(Summary)

  • Remove onNewToken() in ncmbfcmplugin
  • Use queue to handle GetDeviceToken callback

動作確認手順(Step for Confirmation)

  • Setup app using NCMB Unity SDK
  • Check "use push" in NCMBSettings
  • Add code for a GameObject to call GetDeviceToken and show log
    Ex: add bellow code block to start() method

NCMBInstallation ins = NCMBInstallation.getCurrentInstallation();
ins.GetDeviceToken((token, error) =>
{
if(error == null){
Debug.Log("Device Token: " + token);
} else {
Debug.LogError("Error : " + error.Message);
}
});

  • Android Test
    -Run app and check "Device Token: xxx" in log
    -Turn off internet access, run app and check "Error: xxx" in log
  • iOS Test
    -Build, run app and check "Device Token: xxx" in log
    -Turn off "Push Notification" in Xcode Project Setting, build, run app and check "Error: xxx" in log

@vfa-thanhnh vfa-thanhnh force-pushed the fix/update_installationEvents branch 8 times, most recently from d932d64 to 186d8a4 Compare November 23, 2018 10:26
@vfa-thanhnh vfa-thanhnh force-pushed the fix/update_installationEvents branch 2 times, most recently from 8d35a21 to a9e3d88 Compare November 28, 2018 02:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants