Skip to content

Conversation

@miya
Copy link
Member

@miya miya commented Nov 27, 2025

@miya miya requested a review from yuki-takei November 27, 2025 08:46
@miya miya self-assigned this Nov 27, 2025
} satisfies { props: PageProps };

return mergeGetServerSidePropsResults(commonResult, UserGroupDetailPropsFragment);
};
Copy link
Member Author

Choose a reason for hiding this comment

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

getServerSideAdminCommonProps の中には isAclEnabled は含まれていないためユーザーグループ設定の next page の getServerSideProps でとってくるように修正

旧実装のコードを参考

const injectServerConfigurations = async (
context: GetServerSidePropsContext,
props: Props,
): Promise<void> => {
const req: CrowiRequest = context.req as CrowiRequest;
props.isAclEnabled = req.crowi.aclService.isAclEnabled();
};
export const getServerSideProps: GetServerSideProps = async (
context: GetServerSidePropsContext,
) => {
const props = await retrieveServerSideProps(
context,
injectServerConfigurations,
);
return props;
};

@yuki-takei yuki-takei merged commit 78ad419 into dev/7.4.x Nov 27, 2025
16 checks passed
@yuki-takei yuki-takei deleted the fix/174840-specific-element-not-displayed-on-user-group-page branch November 27, 2025 20:26
@github-actions github-actions bot mentioned this pull request Nov 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants