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

NTV-548: Unable to resolve host "api.kickstarter.com" #1632

Merged
merged 3 commits into from May 31, 2022

Conversation

Arkariang
Copy link
Contributor

@Arkariang Arkariang commented May 31, 2022

📲 What

Steps to reproduce:
1 - Get a push notification (Can be one of the generated on internal tools like the project launch as showed in the video)

2 - Deactivate wifi or any networking on the app

3- Kill the app

4 - Open the push notification

5 - hit back button, at this point the app was crashing

🤔 Why

👀 See

| Before 🐛 |

crash.mp4

| After 🦋 |

no_crash.mp4

| | |

Story 📖

NTV-548

- Resolved crash when opening a push notification
@codecov
Copy link

codecov bot commented May 31, 2022

Codecov Report

Merging #1632 (de173b1) into master (2bb8949) will decrease coverage by 0.00%.
The diff coverage is 71.42%.

@@             Coverage Diff              @@
##             master    #1632      +/-   ##
============================================
- Coverage     78.35%   78.34%   -0.01%     
  Complexity     1862     1862              
============================================
  Files           353      353              
  Lines         16509    16502       -7     
  Branches       2031     2032       +1     
============================================
- Hits          12936    12929       -7     
+ Misses         2413     2412       -1     
- Partials       1160     1161       +1     
Impacted Files Coverage Δ
.../kickstarter/libs/rx/operators/ApiErrorOperator.kt 71.42% <71.42%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2bb8949...de173b1. Read the comment docs.

@Arkariang Arkariang marked this pull request as ready for review May 31, 2022 18:17
Copy link
Contributor

@hadia hadia left a comment

Choose a reason for hiding this comment

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

tested and working 👌

@Arkariang Arkariang merged commit 1a34192 into master May 31, 2022
@Arkariang Arkariang deleted the imartin/crash-opening-notification branch May 31, 2022 18:39
Arkariang added a commit that referenced this pull request Jun 6, 2022
…e/NTV-411-integrate-payment-sheet

* 'master' of github.com:kickstarter/android-oss:
  versions and codes
  NTV-552 : Migrate BackingActivity ,BackingViewModel classes to kotlin (#1635)
  NTV-371 :  Migrate `DiscoveryPagerAdapter.java` (#1630)
  NTV 547 : Migrate key classes to kotlin (#1633)
  NTV-546   Migrate qualifiers package Interface to Kotlin (#1631)
  NTV-550: Creator facing crash (#1634)
  NTV-548: Unable to resolve host "api.kickstarter.com" (#1632)
  [no-jira]: Update gradle & kotlin  (#1556)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants