FIX: Required field for Name (PR #12916)

Name must be a required field only if full name is required is enabled in the admin page. The backend works fine, however, in UI the required red asterisk is displayed irrespective of the settings configured.

Fixes 188225

GitHub

This is a bit odd, shouldn’t it be {{#if siteSettings. full_name_required}} to match other templates? I am surprised this works.

@eviltrout Yes, it’s best to match other templates. Have modified this. I believe this.siteSettings works because it’s available through the controller. Please correct me if I am wrong.

The title of this pull request changed from “Required field for Name” to "FIX: Required field for Name

Looks like linting is still failing :frowning:

This pull request has been mentioned on Discourse Meta. There might be relevant details there: