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.

Sign up for GitHub

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

Jump to bottom

Remove HTML tags from error message when creating user in multisite #473

Closed
wants to merge 5 commits into from
Closed

Remove HTML tags from error message when creating user in multisite #473

wants to merge 5 commits into from

Conversation

ernilambar
Copy link
Member

Fixes #472

  • Remove HTML tags from error message when creating user in multisite

Before, error message is like this:

Error: <strong>Error:</strong> This email address is already registered. <a href="http://bahu.local/wp-login.php">Log in</a> with this address or choose another one.

After:

Error: This email address is already registered. Log in with this address or choose another one.

@ernilambar ernilambar requested a review from a team as a code owner February 21, 2024 10:48
swissspidy
swissspidy reviewed Feb 21, 2024
View reviewed changes
src/User_Command.php Outdated Show resolved Hide resolved
@ernilambar
Copy link
Member Author

In WP 3.7 was Sorry, that email address is already used!. So, GitHub actions are not passing.

@ernilambar
Copy link
Member Author

Closing. We could explore more robust approach if needed.

@ernilambar ernilambar closed this Jul 23, 2024
@ernilambar ernilambar deleted the 472-user-create-multisite-error branch July 23, 2024 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@swissspidy swissspidy swissspidy left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Improve message in error trying to create user with existing email in multisite
2 participants
@ernilambar @swissspidy