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.

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

Update Register.vue #1127

Merged
merged 2 commits into from
Aug 21, 2022
Merged

Update Register.vue #1127

merged 2 commits into from
Aug 21, 2022

Conversation

JoshSalway
Copy link
Contributor

@JoshSalway JoshSalway commented Aug 21, 2022

Add 'required' to JetCheckbox 'terms'.

This is required because terms and conditions should be accepted if this feature has been enabled by the users.

Screen Shot 2022-08-21 at 10 08 34 pm

Also the <JetInputError class="mt-2" :message="form.errors.terms" /> should show if there is an error relating to the terms.

Screen Shot 2022-08-21 at 10 15 15 pm

@taylorotwell taylorotwell merged commit 2371d01 into laravel:2.x Aug 21, 2022
taylorotwell added a commit that referenced this pull request Oct 7, 2022
* Register: add required to terms checkbox

livewire was missed in #1127
the error message is already present

* Update register.blade.php

Co-authored-by: Taylor Otwell <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants