Skip to content
This repository has been archived by the owner on Jun 9, 2023. It is now read-only.

Handle exceptions while opening a browser (#108) #193

Merged
merged 4 commits into from
May 4, 2020

Conversation

SubhrajyotiSen
Copy link

No description provided.

@SubhrajyotiSen SubhrajyotiSen force-pushed the browser-check branch 2 times, most recently from 424fca2 to a150947 Compare May 3, 2020 18:35
Copy link
Owner

@Tunous Tunous left a comment

Choose a reason for hiding this comment

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

Please also include appropriate entry in CHANGELOG.md file pointing to this pull request.

* edit CHANGELOG
* use string resource for error message
* move finish() out of try block
@Tunous
Copy link
Owner

Tunous commented May 4, 2020

Almost there but common_error_no_app_found_to_handle_url requires url as single parameter. Should use proper formatting function and pass it before displaying.

@SubhrajyotiSen
Copy link
Author

I should have run the app again🤦 My bad. Will update in some time

@Tunous Tunous merged commit 8444909 into Tunous:master May 4, 2020
@Tunous
Copy link
Owner

Tunous commented May 4, 2020

Great, thanks.

@Tunous Tunous added this to the 0.9.1 milestone May 4, 2020
@Tunous Tunous added the bug Something isn't working label May 4, 2020
msfjarvis added a commit to msfjarvis/Dawn that referenced this pull request Aug 5, 2020
…jarvis/swipe-gesture-icon-position' into features

* msfjarvis/multiple_accounts: (24 commits)
  Adds multiple accounts support
  Prepare next version
  Update changelog for 0.9.1 (Tunous#198)
  Force logout all users (Tunous#196)
  Handle exceptions while opening a browser (Tunous#108) (Tunous#193)
  Prevent selection of negative ranges in SelectionLimitingTextView (fixes Tunous#190) (Tunous#194)
  Prepare next version
  Release 0.9.0 (Tunous#184)
  Release workflow (Tunous#185)
  Fix image loading issues due to http urls (Tunous#183)
  Selftext selection (Tunous#178)
  Submission title selection (Tunous#182)
  Update changelog with missed changes
  Wrap comment byline when it is too long to fit in one line (Tunous#145)
  Revert "Run Debug workflow only on push"
  Upgrade to Gradle 6.3 and add checksum field for F-Droid (Tunous#180)
  Run Debug workflow only on push
  Upgrade Gradle to 6.2.1 (Tunous#152)
  Use GitHub Actions (#72)
  Switch to on-device link unfurler (Tunous#138)
  ...

* msfjarvis/swipe-gesture-icon-position: (26 commits)
  Return false to keep the touch flow going
  Make the gestures' icons follow touch event position on Y
  Swipe Actions' icons at same level as touch event
  Prepare next version
  Update changelog for 0.9.1 (Tunous#198)
  Force logout all users (Tunous#196)
  Handle exceptions while opening a browser (Tunous#108) (Tunous#193)
  Prevent selection of negative ranges in SelectionLimitingTextView (fixes Tunous#190) (Tunous#194)
  Prepare next version
  Release 0.9.0 (Tunous#184)
  Release workflow (Tunous#185)
  Fix image loading issues due to http urls (Tunous#183)
  Selftext selection (Tunous#178)
  Submission title selection (Tunous#182)
  Update changelog with missed changes
  Wrap comment byline when it is too long to fit in one line (Tunous#145)
  Revert "Run Debug workflow only on push"
  Upgrade to Gradle 6.3 and add checksum field for F-Droid (Tunous#180)
  Run Debug workflow only on push
  Upgrade Gradle to 6.2.1 (Tunous#152)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants