Remove register of device on reset.

[mat1th]

This PR does introduce the following:

• Remove the registerDevice call in the reset function. Because it is already called on the configure function.

[alexkok]

Sorry, but we cannot remove the device registration call on the reset() function. This change would result in issues with handling notifications from Parley.

Could you describe why this would be needed in your case? If we know the use case, we can maybe come with a proper solution.

[mat1th]

Why would it result in issues? The push token is already send with the configure, and will also be send in all those situations:

• When Configure is called
• When push is setup (setPushToken and setPushEnabled).
• When device will go to the foreground willEnterForeground All data should already be available on the back-end. The only case I can find is the referrer, but for me that sounds not be necessary to send push messages. What do I miss that is needed to handle the push notifications from parley?

[alexkok]

It could cause users getting push notifications that are not intended for them anymore.

The reset() is intended, for example, when a logout of the user is happening. In that case, we need to update the device with the user to reset the token. Such that the user does not receive push notifications anymore as that user, because the user is logged out.

So, if we remove this device registration call on the reset() method. The user could logout, close the app, but Parley could still trigger push notifications as if the user was logged in, which is unwanted.

Ofcourse, this is solved if the user logins again (and then triggers any of the calls that register the device). But its not certain that these methods are being called after logging in either.

Therefore the question: In what case would a reset be needed without registering the device?

[mat1th]

Thank you for the explanation. We do have the following case:

• App is registered
• User logs in app
• Parley will be started.
• User logs out. This moment we want to clear all user information from memory. But we do not want to de-register push.

This sounds like a different use case. Are you open to add a new function to reset the local user information without resetting push?