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(ce-plugins): Remove or replace deprecated plugins #2419

Merged
merged 2 commits into from
Feb 7, 2020
Merged

docs(ce-plugins): Remove or replace deprecated plugins #2419

merged 2 commits into from
Feb 7, 2020

Conversation

antoninbeaufort
Copy link
Contributor

Hi,

Simply remove deprecated plugins add the category set of plugins with the replacement of the deprecated plugins.

Hi,

Simply remove deprecated plugins add the category set of plugins with the replacement of the deprecated plugins.
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.

Thanks for the PR, but we only add new plugins if the plugin author adds them.

The Fused Location change is ok since it's just a rename, but the @jeepq/capacitor is not just a renaming, is a new plugin (or something like that), so better leave the "Set of plugins" section out.

@antoninbeaufort
Copy link
Contributor Author

Ok if you want, I made this change because it's the author itself that said that the previous plugins was replaced by this set of plugins :

 ***********************************************
  *                                             *
  * This capacitor plugin has been deprecated   *
  *                                             *
  * and added to the Capacitor Plugin Library   *
  *                                             *
  *             @jeepq/capacitor                *
  *                                             *
  ***********************************************

  Development will continue from there

Like said here :

But I will remove this section so.

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.

looks good

@jcesarmobile jcesarmobile changed the title docs(community-plugins) Replace deprecated plugins docs(ce-plugins): Remove or replace deprecated plugins Feb 7, 2020
@jcesarmobile jcesarmobile merged commit dfc1ed6 into ionic-team:master Feb 7, 2020
priyankpat pushed a commit to priyankpat/capacitor that referenced this pull request Feb 13, 2020
chrisweight added a commit to chrisweight/capacitor that referenced this pull request Feb 17, 2020
* upstream/master: (50 commits)
  feat(Device): Add getBatteryInfo function (ionic-team#2435)
  fix(doctor): add electron checks (ionic-team#2434)
  chore(android): target SDK version 29 (ionic-team#2433)
  feat(android): update gradle and dependencies (ionic-team#2431)
  fix(ios): Make Clipboard plugin return errors (ionic-team#2430)
  feat(android): Handle onDestroy lifecycle event in plugins (ionic-team#2421)
  docs(ce-plugins): Remove or replace deprecated plugins (ionic-team#2419)
  feat(ios): change native location accuracy values (ionic-team#2420)
  docs(ce-guides): Remove dead link (ionic-team#2418)
  docs(network): Remove example guide because of dead link (ionic-team#2417)
  feat(electron): Remove injectCapacitor function (ionic-team#2415)
  fix(android): plugin retained events not being retained if listeners were empty (ionic-team#2408)
  chore(android): remove unused launch_splash.xml (ionic-team#2411)
  feat(Filesystem): Remove createIntermediateDirectories from MkdirOptions (ionic-team#2410)
  feat(android): use Fused Location Provider on Geolocation plugin (ionic-team#2409)
  fix(toast): unify duration across platforms (ionic-team#2340)
  feat(PushNotifications): Make register method return if permission was granted (ionic-team#2324)
  chore(circleci): update Xcode and remove install-cocoapods job (ionic-team#2402)
  chore(cli): fix tests for newer node versions (ionic-team#2403)
  fix(android): return original camera image if edition was canceled (ionic-team#2358)
  ...
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.

2 participants