Problem/Motivation

There is no indication on the user check page when no addresses failed the check or when the check was completed.

Proposed resolution

Add some sort of message to indicate that the check was completed, and if no failed addresses were found.

Remaining tasks

  1. Fix the issue
  2. Create a patch
  3. Review the patch
  4. Commit the patch

User interface changes

None.

API changes

None.

Data model changes

None.

Comments

oadaeh created an issue. See original summary.

oadaeh’s picture

Assigned: Unassigned » oadaeh
oadaeh’s picture

Issue summary: View changes
StatusFileSize
new2.32 KB

Attached is the patch to fix this.

oadaeh’s picture

Assigned: oadaeh » Unassigned
oadaeh’s picture

Status: Active » Needs review
kristen pol’s picture

Code looks good. We'll get this tested soon.

Patrick Storey’s picture

Status: Needs review » Reviewed & tested by the community

Before this patch I saw no messages upon success or failure (on failure though it did populate the table in the bottom with the users that failed under the title of "Found 1 user whose email address is not valid.")

Upon adding this patch to the module for a success I received a green message at the top of the form of "Look for the verification results below the form."

The same message occurred when an email was not validated. Except when a users email did not pass validation the table shows the user id that failed and gives the reason why it did. Instead of no users being listed in the table and a message of "All email address checked passed the verification(s)." which is what occurs when all checked users email passed validation.

This has passed testing.

  • oadaeh committed e5c41a1 on 7.x-2.x
    Issue #2669258 by oadaeh, Kristen Pol, Patrick Storey: Add message to...
oadaeh’s picture

Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

This has been committed to the 2.x dev branch. Thanks for the reviews and testing.

kristen pol’s picture

Thanks!

oadaeh’s picture

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.