Skip to content

Commit 45d99a6

Browse files
authored
Role create and edit trigger validation on submit instead of disablin… (#8203)
1 parent fcfecb4 commit 45d99a6

2 files changed

Lines changed: 19 additions & 14 deletions

File tree

.changeset/floppy-houses-play.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'hive': minor
3+
---
4+
5+
Role create and edit trigger validation on submit instead of disabling the submit button

packages/web/app/src/components/organization/members/roles.tsx

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -141,6 +141,10 @@ function OrganizationMemberRoleEditor(props: {
141141
}, []);
142142

143143
async function onSubmit(data: RoleFormValues) {
144+
const isValid = await form.trigger();
145+
if (!isValid) {
146+
return;
147+
}
144148
try {
145149
const result = await updateMemberRole({
146150
input: {
@@ -249,9 +253,7 @@ function OrganizationMemberRoleEditor(props: {
249253
<Button
250254
type="submit"
251255
onClick={form.handleSubmit(onSubmit)}
252-
disabled={
253-
!form.formState.isValid || form.formState.isSubmitting || form.formState.disabled
254-
}
256+
disabled={form.formState.isSubmitting || form.formState.disabled}
255257
>
256258
{form.formState.isSubmitting ? 'Creating...' : 'Confirm selection'}
257259
</Button>
@@ -525,12 +527,14 @@ function OrganizationMemberRoleCreator(props: {
525527
</Button>
526528
<Button
527529
type="submit"
528-
onClick={() => setState('confirm')}
529-
disabled={
530-
!form.formState.isValid ||
531-
form.formState.isSubmitting ||
532-
form.formState.disabled
533-
}
530+
onClick={async () => {
531+
const isValid = await form.trigger();
532+
if (!isValid) {
533+
return;
534+
}
535+
setState('confirm');
536+
}}
537+
disabled={form.formState.isSubmitting || form.formState.disabled}
534538
>
535539
Confirm selection
536540
</Button>
@@ -556,11 +560,7 @@ function OrganizationMemberRoleCreator(props: {
556560
<Button
557561
type="submit"
558562
onClick={form.handleSubmit(onSubmit)}
559-
disabled={
560-
!form.formState.isValid ||
561-
form.formState.isSubmitting ||
562-
form.formState.disabled
563-
}
563+
disabled={form.formState.isSubmitting || form.formState.disabled}
564564
>
565565
{form.formState.isSubmitting
566566
? 'Creating...'

0 commit comments

Comments
 (0)