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

docs(google-maps/geolocation)!: Removing deprecated usage descriptions for iOS #1855

Merged
merged 5 commits into from
Oct 30, 2023

Conversation

theproducer
Copy link
Contributor

@theproducer theproducer commented Oct 23, 2023

Removing NSLocationAlwaysUsageDescription in exchange for NSLocationAlwaysAndWhenInUseUsageDescription as its now deprecated.

@theproducer theproducer marked this pull request as ready for review October 23, 2023 16:51
@jcesarmobile jcesarmobile changed the title docs(google-maps,local-notifications): Updating deprecated privacy description name for iOS docs(google-maps/geolocation): Updating deprecated privacy description name for iOS Oct 23, 2023
@jcesarmobile jcesarmobile changed the title docs(google-maps/geolocation): Updating deprecated privacy description name for iOS docs(google-maps/geolocation)!: Updating deprecated privacy description name for iOS Oct 23, 2023
Copy link
Member

@jcesarmobile jcesarmobile left a comment

Choose a reason for hiding this comment

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

NSLocationAlwaysAndWhenInUseUsageDescription is for when the app gets the location in the background and I don't think the plugin allows that, so I don't think we should add a replacement, just remove the NSLocationAlwaysUsageDescription.
No need to add a note about it being deprecated since it was also not needed.

I've marked it as breaking change so we document in the upgrade guide that users should remove the NSLocationAlwaysUsageDescription if they have it and don't use background geolocation (with another plugin)

@github-actions
Copy link

github-actions bot commented Oct 25, 2023

Released dev build of geolocation with dev version: 5.0.7-dev-1855-20231025T154148.0

@jcesarmobile jcesarmobile changed the title docs(google-maps/geolocation)!: Updating deprecated privacy description name for iOS docs(google-maps/geolocation)!: Removing deprecated usage descriptions for iOS Oct 25, 2023
@theproducer theproducer merged commit 27e7825 into main Oct 30, 2023
15 checks passed
@theproducer theproducer deleted the docs/ios-location-usage branch October 30, 2023 14:51
LaravelFreelancerNL pushed a commit to mister-diels/capacitor-plugins that referenced this pull request Dec 19, 2023
…s for iOS (ionic-team#1855)

Co-authored-by: jcesarmobile <jcesarmobile@gmail.com>
NetEvolutions added a commit to werktools/capacitor-plugins that referenced this pull request Jan 30, 2024
…ols/capacitor-plugins into ios-location-aware-camera

* 'ios-location-aware-camera' of https://github.com/werktools/capacitor-plugins: (24 commits)
  fix(keyboard): distribute the new SPM files (ionic-team#1895)
  chore: only lint plugins that changed (ionic-team#1880)
  feat(google-maps)!: Update iOS dependencies (ionic-team#1890)
  chore: update Podfile paths (ionic-team#1891)
  chore(ios): fix verify script (ionic-team#1892)
  chore(release): publish [skip ci]
  docs(README): Update note for Capacitor 6 (ionic-team#1883)
  chore: update dependencies to use next tag (ionic-team#1889)
  chore: bump version to 6.0.0-alpha.0 (ionic-team#1888)
  feat(app,haptics,status-bar,keyboard): Supporting Swift Package Manager (ionic-team#1886)
  chore(android): upgrade gradle wrapper and plugin to 8.2.x (ionic-team#1885)
  refactor(ios): remove removeAllListeners methods (ionic-team#1867)
  feat(camera)!: Use Android Photo Picker instead of ACTION_PICK intent (ionic-team#1833)
  fix(share): Address Android 14 changes (ionic-team#1849)
  chore: Remove lerna bootstrap form scripts (ionic-team#1875)
  chore: fix toggle-local npm script (ionic-team#1847)
  feat(filesystem): add removeAllListeners method (ionic-team#1862)
  docs(google-maps/geolocation)!: Removing deprecated usage descriptions for iOS (ionic-team#1855)
  chore: use the new addListener return type (ionic-team#1846)
  chore: ci release dev for non-draft PR's (ionic-team#1854)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

3 participants