Skip to content

iOS. There is a crash when a device is registered in FCM.#332

Merged
rdelrosario merged 1 commit intoCrossGeeks:masterfrom
humbertocg:fix-DidReceiveRegistrationToken-crash
Jun 23, 2020
Merged

iOS. There is a crash when a device is registered in FCM.#332
rdelrosario merged 1 commit intoCrossGeeks:masterfrom
humbertocg:fix-DidReceiveRegistrationToken-crash

Conversation

@humbertocg
Copy link
Copy Markdown

✨ What kind of change does this PR introduce?

bugfix

⤵️ What is the current behavior?

There is a crash when a device is registered in FCM

🆕 What is the new behavior (if this is a feature change)?

Avoid the crash by adding a validation

🐛 Recommendations for testing

  1. The user setup to receive push notifications.
  2. The user doesn't want to receive push notifications (unregister him device from FCM)
  3. And then the user tries to register again him device in FCM

The pendingTopics stack has some values in null.

🤔 Checklist before submitting

  • [ X] All projects build
  • Follows style guide lines
  • Relevant documentation was updated
  • Rebased onto current develop

1. The user setup to receive push notifications.
2. The user doesn't want to receive push notifications (unregister him device from FCM)
3. And then the user tries to register again him device in FCM

the pendingTopics stack has some values in null.
Copy link
Copy Markdown
Member

@rdelrosario rdelrosario left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

@rdelrosario rdelrosario merged commit fd9d2c4 into CrossGeeks:master Jun 23, 2020
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