U4-10004 - Healthchecks that allow you to provide a value to fix the issue only allow values in the form of email addresses

Created by Andy Butland 10 Jun 2017, 21:33:57 Updated by Tom Pipe 14 Mar 2018, 18:44:24

If you look [here|https://github.com/umbraco/Umbraco-CMS/blob/dev-v7/src/Umbraco.Web.UI.Client/src/views/dashboard/developer/healthcheck.html#L104] you can see the field where you enter the correct value in has a val-email attribute. Which is correct for the core health check for "Notification Email Settings", as that should be an email. But validation for an email should only be for that check, not for all.

Expect it's not been noted before as I believe the "Notification Email Settings" check is the only one in core that allows you to provide a "fix" value.

See https://our.umbraco.org/forum/extending-umbraco-and-using-the-api/86270-health-check-fix-button-is-disabled# for more details.

Comments

Andy Butland 11 Jun 2017, 21:47:41

PR here: https://github.com/umbraco/Umbraco-CMS/pull/1994


Tom Pipe 14 Mar 2018, 18:44:24

Seeing this issue again today with one of our health checks, disappointed to see it's still hanging around.

@sebastiaan - any chance of getting this merged in?


Priority: Normal

Type: Bug

State: Submitted

Assignee:

Difficulty: Normal

Category:

Backwards Compatible: True

Fix Submitted:

Affected versions: 7.6.3

Due in version:

Sprint:

Story Points:

Cycle: