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

Replaced usage of NoPermissionsFragment with AlertDialog in NearbyActivity #866

Conversation

Bluesir9
Copy link
Contributor

@Bluesir9 Bluesir9 commented Sep 5, 2017

The usage of NoPermissionsFragment when location run-time permission has been denied in NearbyActivity has been replaced with the usage of an AlertDialog providing the user actions to either retry the run-time permission again or cancel the request(in which case the user exits the NearbyActivity).

… of the NoPermissionsFragment, now an AlertDialog is shown providing options to either retry the permission request or cancel and quit the activity.
@Bluesir9
Copy link
Contributor Author

Bluesir9 commented Sep 5, 2017

A screenshot of what the AlertDialog looks like:

screenshot_1504624444

@codecov-io
Copy link

Codecov Report

Merging #866 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master   #866      +/-   ##
========================================
- Coverage    6.91%   6.9%   -0.02%     
========================================
  Files          95     95              
  Lines        5018   5028      +10     
  Branches      471    471              
========================================
  Hits          347    347              
- Misses       4644   4654      +10     
  Partials       27     27
Impacted Files Coverage Δ
...ava/fr/free/nrw/commons/nearby/NearbyActivity.java 37.9% <0%> (-2.66%) ⬇️

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 d4a89af...11f9511. Read the comment docs.

@misaochan
Copy link
Member

Sorry it took so long to get back to you, @Bluesir9 ! I'm good with merging this once the conflicts have been resolved.

@Bluesir9
Copy link
Contributor Author

@misaochan Have resolved the conflicts found.

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.

3 participants