feat(article): prefer to fetch article detail data by articleId inste…

…ad of mediaHash
thematters · Oct 10, 2023 · c0003a6 · c0003a6
1 parent 18ab421
commit c0003a6
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/src/views/ArticleDetail/index.tsx b/src/views/ArticleDetail/index.tsx
@@ -380,7 +380,11 @@ const ArticleDetail = ({
   /**
    * fetch public data
    */
-  const isQueryByHash = !!(mediaHash && isMediaHashPossiblyValid(mediaHash))
+  const isQueryByHash = !!(
+    mediaHash &&
+    isMediaHashPossiblyValid(mediaHash) &&
+    !articleId
+  )
 
   // backward compatible with:
   // - `/:username:/:articleId:-:slug:-:mediaHash`
@@ -607,7 +611,12 @@ const ArticleDetailOuter = () => {
   const articleId =
     (router.query.mediaHash as string)?.match(/^(\d+)/)?.[1] || ''
 
-  const isQueryByHash = !!(mediaHash && isMediaHashPossiblyValid(mediaHash))
+  const isQueryByHash = !!(
+    mediaHash &&
+    isMediaHashPossiblyValid(mediaHash) &&
+    !articleId
+  )
+
   const resultByHash = usePublicQuery<ArticleAvailableTranslationsQuery>(
     ARTICLE_AVAILABLE_TRANSLATIONS,
     { variables: { mediaHash }, skip: !isQueryByHash }