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

fix: Phone state breadcrumbs require read_phone_state on older OS versions #415

Merged
merged 2 commits into from
May 16, 2020

Conversation

marandaneto
Copy link
Contributor

@marandaneto marandaneto commented May 16, 2020

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

fix: Phone state breadcrumbs require read_phone_state on older OS versions

💡 Motivation and Context

Fix: #413

💚 How did you test it?

running API 14 and 16

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • All tests passing

🔮 Next steps

@marandaneto marandaneto changed the title fix/phone state permission fix: Phone state breadcrumbs require read_phone_state on older OS versions May 16, 2020
@codecov-io
Copy link

codecov-io commented May 16, 2020

Codecov Report

Merging #415 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #415   +/-   ##
=========================================
  Coverage     60.18%   60.18%           
  Complexity      809      809           
=========================================
  Files            92       92           
  Lines          3747     3747           
  Branches        360      360           
=========================================
  Hits           2255     2255           
  Misses         1338     1338           
  Partials        154      154           

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 8b964c6...18dae94. Read the comment docs.

@marandaneto marandaneto merged commit 88c87d4 into master May 16, 2020
@marandaneto marandaneto deleted the fix/phone-state-permission branch May 16, 2020 07:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants