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

Minor fit and finish issues, primarily on iOS, but one for android #821

Merged
merged 3 commits into from
Apr 27, 2022

Conversation

shankari
Copy link
Contributor

  • iOS: fix crash in the transition notifier + convert all asserts to checks to avoid future crashes
  • iOS: fix "notch" issue for recent versions of the iPhone
  • iOS: ensure that we don't complain if we get more notification permissions than we asked for
  • iOS + android: fix/workaround the "duplicate entries" issue by turning on the loading overlay so users can't click any other buttons while we are communicating with the server

This fixes e-mission/e-mission-docs#722 and e-mission/e-mission-docs#658

This fixes the second issue in
e-mission/e-mission-docs#722 (comment)

In particular, we add the `viewport-fit=cover` tag.
Unfortunately, since we are using ionic version 1, it doesn't automatically set
all the safe areas correctly.

So we had to figure out how to send them manually.
Finally got it to work, applied it only to iOS
e-mission/e-mission-docs#722 (comment)

Both iOS and android work well now, at least in profile mode
e-mission/e-mission-docs#722 (comment)
- notification check: e-mission-data-collection
    e-mission/e-mission-docs#722 (comment)
- crashes e-mission-transition-notify, cordova-usercache
    e-mission/e-mission-docs#722 (comment)
…reen

This fixes
e-mission/e-mission-docs#658

Thanks to @allenmichael099 for reporting a reproducible use case that I could
fix in under an hour
@shankari shankari merged commit c4e5972 into e-mission:master Apr 27, 2022
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.

Couple of native code fixes
1 participant