Skip to content

FIX: Show tip for required selectable field on signup when not selected #33401

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Lhcfl
Copy link
Contributor

@Lhcfl Lhcfl commented Jul 1, 2025

Dropdown and multiselect components lack InputTip, which makes them not show any reason when validation fails.

This commit also adds a new i18n message for select fields, after this commit, if a multiselect or dropdown custom field required has no option selected, it will display a Please select a value for "XX" field validation error when the Signup button is clicked.

Screenshot

image

Dropdown and multiselect components lack `InputTip`, which makes them
not show any reason when validation fails.

This commit also adds a new i18n message for select fields, after this
commit, if a multiselect or dropdown custom field required has no option
selected, it will display a `Please select a value for "XX" field`
validation error when the Signup button is clicked.
@github-actions github-actions bot added the i18n PRs which update English locale files or i18n related code label Jul 1, 2025
@Lhcfl Lhcfl changed the title FIX: Show tip for required selectable field on signin when not selected FIX: Show tip for required selectable field on signup when not selected Jul 1, 2025
@@ -3309,7 +3310,7 @@ en:
hot: "There are no hot topics."
filter: "There are no topics."
education:
topic_tracking_preferences: "You can view and change your new topic tracking settings in <a href=\"%{basePath}/my/preferences/tracking\">your preferences</a>."
topic_tracking_preferences: 'You can view and change your new topic tracking settings in <a href="%{basePath}/my/preferences/tracking">your preferences</a>.'
Copy link
Contributor

@nattsw nattsw Jul 1, 2025

Choose a reason for hiding this comment

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

I think you can undo the change here in this PR since it’s unrelated and no difference. (though it is better imo)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i18n PRs which update English locale files or i18n related code
Development

Successfully merging this pull request may close these issues.

2 participants