Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix forms preview in our own apps #614

Merged
merged 2 commits into from
Mar 5, 2018
Merged

Conversation

guperrot
Copy link
Member

@guperrot guperrot commented Mar 3, 2018

No description provided.

@guperrot guperrot requested a review from achocron March 3, 2018 01:31
@rizamarhaban
Copy link

rizamarhaban commented Mar 5, 2018

This works! However, the Push notification event did not fired if the app is killed. If the app is onsleep, this works.

@guperrot guperrot merged commit 320db33 into develop Mar 5, 2018
@guperrot guperrot deleted the fix/test-apps-forms-previewer branch March 5, 2018 19:13
@Haidar-Hammoud
Copy link

I'm sorry. Does this actually fix this issue?? I'm using Android Studio, is it worth plopping this into my project?

@guperrot
Copy link
Member Author

guperrot commented Mar 6, 2018

@HAIDARCORP this is a Xamarin PR just to change the implementation of "avoid duplicate push event registration" in a way that does not break the previewer and is very specific to C#.

Android Studio is for Java/Kotlin/C/C++ only and does not have duplicate issues in Java SDK.

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.

4 participants