Merge pull request #545 from cloudnativedaysjp/feat/include_talkid_in…

…_mutate feat: Include talkID in viewCount
cloudnativedaysjp · Feb 17, 2025 · 54aa29f · 54aa29f
2 parents b50e50d + b904b4f
commit 54aa29f
Show file tree

Hide file tree

Showing 5 changed files with 36 additions and 7 deletions.
diff --git a/src/__generated__/gql.ts b/src/__generated__/gql.ts
diff --git a/src/__generated__/graphql.ts b/src/__generated__/graphql.ts
diff --git a/src/components/TalkInfo/TalkInfo.tsx b/src/components/TalkInfo/TalkInfo.tsx
@@ -23,6 +23,7 @@ export const TalkInfo: React.FC<Props> = ({
     eventAbbr,
     profileId,
     selectedTrack?.name,
+    selectedTalk?.id,
   )
   useEffect(() => {
     return () => {

diff --git a/src/components/hooks/__tests__/useViewerCount.spec.tsx b/src/components/hooks/__tests__/useViewerCount.spec.tsx
@@ -14,6 +14,7 @@ describe.only('useViewerCount', () => {
       profileId: 42,
       trackName: 'B',
       count: 5,
+      talkID: 10,
     }
     const got = {
       count: 0,
@@ -58,6 +59,7 @@ describe.only('useViewerCount', () => {
         given.confName,
         given.profileId,
         given.trackName,
+        given.talkID,
       )
       got.count = count
       if (!!timer && !!count) {

diff --git a/src/components/hooks/useViewerCount.ts b/src/components/hooks/useViewerCount.ts
@@ -7,6 +7,7 @@ export const useViewerCount = (
   eventAbbr: string,
   profileId: number | undefined,
   selectedTrackName: string | undefined,
+  selectedTalkId: number | undefined,
 ): [number, NodeJS.Timeout | null] => {
   const [viewerCounts, setViewerCounts] = useState<Record<string, number>>({})
   const [curTrack, setCurTrack] = useState<string>('')
@@ -29,8 +30,8 @@ export const useViewerCount = (
   })
 
   const viewTrack = gql(`
-    mutation ViewTrack($profileID: Int!, $trackName: String!) {
-      viewTrack(input: { profileID: $profileID, trackName: $trackName })
+    mutation ViewTrack($profileID: Int!, $trackName: String!, $talkID: Int!) {
+      viewTrack(input: { profileID: $profileID, trackName: $trackName, talkID: $talkID })
     }
   `)
   useEffect(() => {
@@ -56,7 +57,7 @@ export const useViewerCount = (
   const [setViewTrack, { error: viewTrackError }] = useMutation(viewTrack)
 
   useEffect(() => {
-    if (!profileId || !selectedTrackName) {
+    if (!profileId || !selectedTrackName || !selectedTalkId) {
       return
     }
     if (selectedTrackName === curTrack) {
@@ -71,14 +72,15 @@ export const useViewerCount = (
         variables: {
           profileID: profileId,
           trackName: selectedTrackName,
+          talkID: selectedTalkId,
         },
       })
     }
     mutate()
     setTimer(setInterval(mutate, 30 * 1000))
 
     setCurTrack(selectedTrackName)
-  }, [selectedTrackName, profileId, timer, curTrack])
+  }, [selectedTrackName, selectedTalkId, profileId, timer, curTrack])
 
   useEffect(() => {
     if (viewTrackError) {