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

Improve error log #530

Merged
merged 1 commit into from
Sep 13, 2018
Merged

Improve error log #530

merged 1 commit into from
Sep 13, 2018

Conversation

joshcanhelp
Copy link
Contributor

  • Add ability to clear out the error log in wp-admin
  • Add get method to WP_Auth0_ErrorLog to retrieve current log
  • Add add method to WP_Auth0_ErrorLog to combine duplicate errors and add time
  • Add clear method to WP_Auth0_ErrorLog to empty the error log
  • Add delete method to WP_Auth0_ErrorLog to delete the error log option
  • Minor CSS imrovements

@@ -21,7 +21,7 @@
background-color: bgColor;

.a0-wrap {
font-family:'avenir roman';
font-family:'avenir roman', "Open Sans", sans-serif;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fallback font to WP default instead of serif

@@ -903,8 +903,8 @@
}

.a0-table {
margin:30px;
width: initial;
margin: 2em 2%;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Add a little breathing room around the admin columns on mobile.

@@ -31,24 +31,21 @@ public function init_menu() {
exit;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

General clean-up

}
} else {
$message = is_object( $error ) || is_array( $error ) ? serialize( $error ) : $error;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Moved all of the actual logging logic into WP_Auth0_ErrorLog::add

@@ -1,41 +1,62 @@
<?php
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Clean-up, output sanitization, and add the count column

@@ -1,8 +1,13 @@
<?php
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Offline support for the header image

@@ -0,0 +1,242 @@
<?php
Copy link
Contributor Author

Choose a reason for hiding this comment

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

🎉

Copy link
Member

@cocojoe cocojoe left a comment

Choose a reason for hiding this comment

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

👍

@joshcanhelp joshcanhelp merged commit 3705ca9 into dev Sep 13, 2018
@joshcanhelp joshcanhelp deleted the improve-error-log branch September 20, 2018 17:36
@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.

2 participants