Skip to content

Commit ede5c6b

Browse files
authored
Merge pull request #760 from BigRoma/main
fix: forcefully rendering child components Closes: #454
2 parents 88118ff + 737cb4b commit ede5c6b

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

src/create.tsx

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -80,8 +80,8 @@ const create = <Role extends string, Permission extends string, User>() => {
8080
const AllowedTo = ({
8181
perform = [],
8282
children,
83-
yes: Yes = () => <React.Fragment>{children}</React.Fragment>,
84-
no: No = () => null,
83+
yes: Yes,
84+
no: No,
8585
data,
8686
}: AllowedToProps<Permission>) => {
8787
const ctx = useAbac();
@@ -93,16 +93,15 @@ const create = <Role extends string, Permission extends string, User>() => {
9393
return null;
9494
}
9595

96-
return ctx.userHasPermissions(ensureArray(perform), data) ? (
97-
<Yes />
98-
) : (
99-
<No />
100-
);
96+
if (ctx.userHasPermissions(ensureArray(perform), data)) {
97+
return Yes ? <Yes /> : <React.Fragment>{children}</React.Fragment>;
98+
}
99+
100+
return No ? <No /> : null;
101101
};
102102

103103
const NotAllowedTo = ({
104-
children,
105-
yes = () => <React.Fragment>{children}</React.Fragment>,
104+
yes,
106105
no,
107106
...props
108107
}: AllowedToProps<Permission>) => (

0 commit comments

Comments
 (0)