admin/bodyshop
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

IO-2950 - Add in double subscription fix

Browse Source 
Signed-off-by: Dave Richer <dave@imexsystems.ca>
This commit is contained in:
Dave Richer
2024-09-19 16:09:49 -04:00
parent 9a0674f5d7 f7af3b407b
commit 56a580b1e7
1 changed files with 11 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
client/src/components/production-board-kanban/production-board-kanban.container.jsx
View File

@@ -1,4 +1,4 @@
import React, { useEffect, useMemo } from "react"; import React, { useEffect, useMemo, useRef } from "react";
import { useQuery, useSubscription } from "@apollo/client"; import { useQuery, useSubscription } from "@apollo/client";
import { connect } from "react-redux"; import { connect } from "react-redux";
import { createStructuredSelector } from "reselect"; import { createStructuredSelector } from "reselect";
@@ -17,6 +17,8 @@ const mapStateToProps = createStructuredSelector({
}); });
function ProductionBoardKanbanContainer({ bodyshop, currentUser, subscriptionType = "direct" }) { function ProductionBoardKanbanContainer({ bodyshop, currentUser, subscriptionType = "direct" }) {
const fired = useRef(false); // useRef to keep track of whether the subscription fired
const combinedStatuses = useMemo( const combinedStatuses = useMemo(
() => [ () => [
...bodyshop.md_ro_statuses.production_statuses, ...bodyshop.md_ro_statuses.production_statuses,
@@ -47,10 +49,15 @@ function ProductionBoardKanbanContainer({ bodyshop, currentUser, subscriptionTyp
// const currentReducerData = useSelector((state) => (state.trello.lanes ? state.trello : {})); // const currentReducerData = useSelector((state) => (state.trello.lanes ? state.trello : {}));
useEffect(() => { useEffect(() => {
if (updatedJobs && data) { if (!updatedJobs) {
refetch().catch((err) => console.error(`Error re-fetching jobs in production: ${err.message}`)); return;
} }
}, [updatedJobs, data, refetch]); if (!fired.current) {
fired.current = true;
return;
}
refetch().catch((err) => console.error(`Error re-fetching jobs in production: ${err.message}`));
}, [updatedJobs, refetch]);
const filteredAssociationSettings = useMemo(() => { const filteredAssociationSettings = useMemo(() => {
return associationSettings?.associations[0] || null; return associationSettings?.associations[0] || null;