Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Deprecate WP_Auth0_UsersRepo methods #652

Merged
merged 1 commit into from
Apr 1, 2019

Conversation

joshcanhelp
Copy link
Contributor

Changes

This PR deprecates the following methods:

  • WP_Auth0_UsersRepo::init() - still used but will be removed in the next major.
  • WP_Auth0_UsersRepo::getUser() - no longer used
  • WP_Auth0_UsersRepo::tokenHasRequiredScopes() - no longer used

@joshcanhelp joshcanhelp added this to the 3.10.0 milestone Mar 30, 2019
@codecov-io
Copy link

Codecov Report

Merging #652 into master will increase coverage by 0.31%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #652      +/-   ##
============================================
+ Coverage     42.12%   42.44%   +0.31%     
+ Complexity     1238     1226      -12     
============================================
  Files            51       51              
  Lines          3779     3751      -28     
============================================
  Hits           1592     1592              
+ Misses         2187     2159      -28
Impacted Files Coverage Δ Complexity Δ
lib/WP_Auth0_Configure_JWTAUTH.php 0% <ø> (ø) 21 <0> (ø) ⬇️
lib/WP_Auth0_UsersRepo.php 66.66% <ø> (+18.66%) 29 <0> (-12) ⬇️

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 4cab340...04063f8. Read the comment docs.

@@ -3,14 +3,14 @@
/**
* Class WP_Auth0_Configure_JWTAUTH
*
* @deprecated 3.10.0, plugin is deprecated and removed from the WP plugin repo.
* @deprecated - 3.10.0, plugin is deprecated and removed from the WP plugin repo.
Copy link
Member

Choose a reason for hiding this comment

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

Is the - a formatting requirement ? I'll approve anyway, just curious.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, just inconsistent with the rest

@joshcanhelp joshcanhelp merged commit 8f54edc into master Apr 1, 2019
@joshcanhelp joshcanhelp deleted the deprecations-for-usersrepo branch April 1, 2019 16:49
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 19, 2022
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants