0
Fork 0
mirror of https://github.com/logto-io/logto.git synced 2025-02-17 22:04:19 -05:00

fix(console): fix AuditLogTable components application selector (#4586)

This commit is contained in:
Darcy Ye 2023-09-27 15:09:01 +08:00 committed by GitHub
parent 65b68a91f2
commit 21a5b4da6c
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -40,11 +40,13 @@ function AuditLogTable({ applicationId, userId, className }: Props) {
useSearchParametersWatcher({
page: 1,
event: '',
...conditional(applicationId && { applicationId: '' }),
// If `applicationId` not specified when init this component, then search parameter of `applicationId` can be accepted.
...conditional(!applicationId && { applicationId: '' }),
});
// TODO: LOG-7135, revisit this fallback logic and see whether this should be done outside of this component.
const searchApplicationId = applicationId ?? applicationIdFromSearch;
// eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
const searchApplicationId = applicationId || applicationIdFromSearch;
const { data: specifiedApplication } = useSWR<ApplicationResponse>(
applicationId && `api/applications/${applicationId}`
);
@ -127,9 +129,12 @@ function AuditLogTable({ applicationId, userId, className }: Props) {
{!applicationId && (
<div className={styles.applicationSelector}>
<ApplicationSelector
value={applicationId}
onChange={(applicationId) => {
updateSearchParameters({ applicationId, page: undefined });
value={applicationIdFromSearch}
onChange={(applicationIdFromSearch) => {
updateSearchParameters({
applicationId: applicationIdFromSearch,
page: undefined,
});
}}
/>
</div>