From 35a8368dfec4ef5a477f1c3aeb8c04438844da38 Mon Sep 17 00:00:00 2001 From: Dennis Benz Date: Tue, 4 Jun 2024 17:00:37 +0200 Subject: [PATCH] Remove race condition in table filters --- src/components/configuration/Themes.tsx | 4 ++-- src/components/events/Events.tsx | 4 ++-- src/components/events/Series.tsx | 4 ++-- src/components/recordings/Recordings.tsx | 4 ++-- src/components/systems/Jobs.tsx | 4 ++-- src/components/systems/Servers.tsx | 4 ++-- src/components/systems/Services.tsx | 4 ++-- src/components/users/Groups.tsx | 4 ++-- src/components/users/Users.tsx | 4 ++-- 9 files changed, 18 insertions(+), 18 deletions(-) diff --git a/src/components/configuration/Themes.tsx b/src/components/configuration/Themes.tsx index f9dd300cc6..fb88d85bcf 100644 --- a/src/components/configuration/Themes.tsx +++ b/src/components/configuration/Themes.tsx @@ -40,8 +40,8 @@ const Themes = ({ const themes = useAppSelector(state => getTotalThemes(state)); // TODO: Get rid of the wrappers when modernizing redux is done - const fetchThemesWrapper = () => { - dispatch(fetchThemes()) + const fetchThemesWrapper = async () => { + await dispatch(fetchThemes()) } const loadThemes = async () => { diff --git a/src/components/events/Events.tsx b/src/components/events/Events.tsx index a6f08b4266..986760014f 100644 --- a/src/components/events/Events.tsx +++ b/src/components/events/Events.tsx @@ -82,8 +82,8 @@ const Events = ({ let location = useLocation(); // TODO: Get rid of the wrappers when modernizing redux is done - const fetchEventsWrapper = () => { - dispatch(fetchEvents()) + const fetchEventsWrapper = async () => { + await dispatch(fetchEvents()) } const loadEvents = async () => { diff --git a/src/components/events/Series.tsx b/src/components/events/Series.tsx index 4638fdd483..bdd7d4de65 100644 --- a/src/components/events/Series.tsx +++ b/src/components/events/Series.tsx @@ -64,8 +64,8 @@ const Series = ({ const showActions = useAppSelector(state => isShowActions(state)); // TODO: Get rid of the wrappers when modernizing redux is done - const fetchSeriesWrapper = () => { - dispatch(fetchSeries()) + const fetchSeriesWrapper = async () => { + await dispatch(fetchSeries()) } const loadEvents = () => { diff --git a/src/components/recordings/Recordings.tsx b/src/components/recordings/Recordings.tsx index 7efc548805..b724c1e780 100644 --- a/src/components/recordings/Recordings.tsx +++ b/src/components/recordings/Recordings.tsx @@ -36,8 +36,8 @@ const Recordings = ({ const recordings = useAppSelector(state => getTotalRecordings(state)); // TODO: Get rid of the wrappers when modernizing redux is done - const fetchRecordingsWrapper = () => { - dispatch(fetchRecordings(undefined)) + const fetchRecordingsWrapper = async () => { + await dispatch(fetchRecordings(undefined)) } const loadRecordings = async () => { diff --git a/src/components/systems/Jobs.tsx b/src/components/systems/Jobs.tsx index 1932bc9a19..fd49c8742d 100644 --- a/src/components/systems/Jobs.tsx +++ b/src/components/systems/Jobs.tsx @@ -49,8 +49,8 @@ const Jobs = ({ const jobs = useAppSelector(state => getTotalJobs(state)); // TODO: Get rid of the wrappers when modernizing redux is done - const fetchJobsWrapper = () => { - dispatch(fetchJobs()) + const fetchJobsWrapper = async () => { + await dispatch(fetchJobs()) } const loadJobs = async () => { diff --git a/src/components/systems/Servers.tsx b/src/components/systems/Servers.tsx index 4e62f04009..f4393b930b 100644 --- a/src/components/systems/Servers.tsx +++ b/src/components/systems/Servers.tsx @@ -49,8 +49,8 @@ const Servers = ({ const servers = useAppSelector(state => getTotalServers(state)); // TODO: Get rid of the wrappers when modernizing redux is done - const fetchServersWrapper = () => { - dispatch(fetchServers()) + const fetchServersWrapper = async () => { + await dispatch(fetchServers()) } const loadServers = async () => { diff --git a/src/components/systems/Services.tsx b/src/components/systems/Services.tsx index 07766f7e4f..fe59c712eb 100644 --- a/src/components/systems/Services.tsx +++ b/src/components/systems/Services.tsx @@ -49,8 +49,8 @@ const Services = ({ const services = useAppSelector(state => getTotalServices(state)); // TODO: Get rid of the wrappers when modernizing redux is done - const fetchServicesWrapper = () => { - dispatch(fetchServices()) + const fetchServicesWrapper = async () => { + await dispatch(fetchServices()) } const loadServices = async () => { diff --git a/src/components/users/Groups.tsx b/src/components/users/Groups.tsx index d2ff6b8018..2093313a8e 100644 --- a/src/components/users/Groups.tsx +++ b/src/components/users/Groups.tsx @@ -51,8 +51,8 @@ const Groups = ({ const currentFilterType = useAppSelector(state => getCurrentFilterResource(state)); // TODO: Get rid of the wrappers when modernizing redux is done - const fetchGroupsWrapper = () => { - dispatch(fetchGroups()) + const fetchGroupsWrapper = async () => { + await dispatch(fetchGroups()) } const loadGroups = async () => { diff --git a/src/components/users/Users.tsx b/src/components/users/Users.tsx index bde1f47b43..563a5de512 100644 --- a/src/components/users/Users.tsx +++ b/src/components/users/Users.tsx @@ -41,8 +41,8 @@ const Users: React.FC = () => { const currentFilterType = useAppSelector(state => getCurrentFilterResource(state)); // TODO: Get rid of the wrappers when modernizing redux is done - const fetchUsersWrapper = () => { - dispatch(fetchUsers()) + const fetchUsersWrapper = async () => { + await dispatch(fetchUsers()) } const loadUsers = async () => {