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

Deprecate methods that control field rendering and validation #659

Merged
merged 3 commits into from
Apr 1, 2019

Conversation

joshcanhelp
Copy link
Contributor

@joshcanhelp joshcanhelp commented Mar 31, 2019

Changes

This PR deprecates the following methods:

  • WP_Auth0_Admin_Advanced::render_link_auth0_users() - Setting no longer used, field will be removed in the next major.
  • WP_Auth0_Admin_Advanced::render_jwt_auth_integration() - Integration is deprecated (Deprecate WP JWT Auth plugin configuration #636).
  • WP_Auth0_Admin_Advanced::link_accounts_validation() - No longer used.
  • WP_Auth0_Admin_Appearance::render_custom_css() - Setting is deprecated and will be removed in the next major. See this docs page for how to use external stylesheets.
  • WP_Auth0_Admin_Appearance::render_custom_js() - Setting is deprecated and will be removed in the next major. See this docs page for how to use external JS.
  • WP_Auth0_Admin_Basic::render_auth0_app_token() - Setting is no longer used.
  • WP_Auth0_Admin_Features::render_password_policy() - Setting is no longer used.
  • WP_Auth0_Admin_Features::render_fullcontact() - Setting is no longer used.
  • WP_Auth0_Admin_Features::render_fullcontact_apikey() - Setting is no longer used.
  • WP_Auth0_Admin_Features::render_geo() - Setting is no longer used.
  • WP_Auth0_Admin_Features::render_income() - Setting is no longer used.
  • WP_Auth0_Admin_Features::sso_validation() - Validation is no longer needed.
  • WP_Auth0_Admin_Features::security_validation() - Validation is no longer needed.
  • WP_Auth0_Admin_Features::fullcontact_validation() - Validation is no longer needed.
  • WP_Auth0_Admin_Features::mfa_validation() - Validation is no longer needed.
  • WP_Auth0_Admin_Features::georule_validation() - Validation is no longer needed.
  • WP_Auth0_Admin_Features::incomerule_validation() - Validation is no longer needed.
  • WP_Auth0_Admin_Generic::rule_validation() - Validation is no longer needed.

@joshcanhelp joshcanhelp added this to the 3.10.0 milestone Mar 31, 2019
@auth0 auth0 deleted a comment from codecov-io Mar 31, 2019
@auth0 auth0 deleted a comment from codecov-io Mar 31, 2019
@codecov-io
Copy link

codecov-io commented Mar 31, 2019

Codecov Report

Merging #659 into master will increase coverage by 0.29%.
The diff coverage is 50%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #659      +/-   ##
============================================
+ Coverage     42.12%   42.42%   +0.29%     
+ Complexity     1238     1235       -3     
============================================
  Files            51       51              
  Lines          3779     3755      -24     
============================================
+ Hits           1592     1593       +1     
+ Misses         2187     2162      -25
Impacted Files Coverage Δ Complexity Δ
lib/admin/WP_Auth0_Admin_Generic.php 72.03% <ø> (ø) 41 <0> (ø) ⬇️
lib/admin/WP_Auth0_Admin_Appearance.php 14.75% <ø> (ø) 19 <0> (ø) ⬇️
lib/admin/WP_Auth0_Admin_Advanced.php 46.95% <ø> (+0.78%) 66 <0> (-1) ⬇️
lib/admin/WP_Auth0_Admin_Basic.php 36.07% <ø> (+1.73%) 29 <0> (-1) ⬇️
lib/admin/WP_Auth0_Admin_Features.php 29.48% <50%> (+2.54%) 33 <0> (-1) ⬇️

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...5bafd64. Read the comment docs.

@joshcanhelp joshcanhelp marked this pull request as ready for review March 31, 2019 14:36
@joshcanhelp joshcanhelp merged commit 8a290a8 into master Apr 1, 2019
@joshcanhelp joshcanhelp deleted the deprecations-for-admin-settings branch April 1, 2019 15:59
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants