Firing receiveTransition if the app is started within a fence

[gleerman]

Hi,

I noticed that when I start my app inside a fence, the receiveTransition is not fired for that fence. I suppose there are some valid arguments for this behaviour, since there is no real transition happening. However, there are cases where it is desirable to notify the app when it is inside a fence on startup.

For iOS, I found this thread. I'm proposing the approach where a check on initialization is implemented. We could provide an argument to the initialize method to toggle this behaviour. Let me know what you think.

Since I am only testing on iOS for now, I am not really aware of the behaviour on Android. I came accross the GEOFENCE_TRANSITION_DWELL transition type, but I am not entirely sure if that is what we are looking for. Anyone who knows more about this for Android or who is willing to test?

Please share your thoughts on this.

[gleerman]

Just did an Android test myself. Turns out the receiveTransition is fired by default on startup. I adjusted the iOS code myself and will commit the fix.

[tsubik]

Hmm, for me it was undesired behaviour. Maybe it would be better to introduce this as an option. People will have a choice if they want to get this very first notification if they already in the fence. Or just add requestStateForRegion as a new function to the plugin.

[gleerman]

At first I would have agreed to introduce this as an option. However, since Android has this behaviour enabled by default, it looks like an inconsistency in the plugin. Since I don't think it's possible to disable this behaviour on Android, I'm not sure what the option should do on the Android side.

[tsubik]

I could add some timestamp to geofence like created_at and just filter out the new ones while creating notification. I am not aware about function like requestStateForRegion for android, though.

[rlataguerra]

looks interesting to me,
+1

[johnrobertcobbold]

+1

[tsubik]

Thiniking about this again. Looks like guys from google added something like that into API. Just wondering about the scope, I think it would be better to have a possibility to set this for single geofence.

[johnrobertcobbold]

Just submitted the iOS check on geofence creation/edit here, based on the latest master version of this plugin @ #125

[mhfen]

The lack of this functionality is causing Android builds to Crash. #158