Merge pull request #33 from line/fix-stp-call

Fix bugs related to ShareTargetPicker
line · Aug 16, 2024 · 6365d1a · 6365d1a · github-actions · Aug 16, 2024
2 parents 6674c5f + 3cb17d2
commit 6365d1a
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/src/App.tsx b/src/App.tsx
@@ -230,7 +230,7 @@ function App() {
           pulldownOptions={SHARE_TARGET_PICKER_FIXED_ARGUMENT_LIST}
           skipAutoRun={true}
           runner={async (options) => {
-            return await liff.shareTargetPicker(JSON.parse(options))
+            return await liff.shareTargetPicker([JSON.parse(options)])
           }}
           loginRequired={true}
         />

diff --git a/src/constants.ts b/src/constants.ts
@@ -1,3 +1,4 @@
+const base = new URL(location.href).origin;
 export const SHARE_TARGET_PICKER_FIXED_ARGUMENT_LIST = [
   {
     label: 'text',
@@ -18,24 +19,24 @@ export const SHARE_TARGET_PICKER_FIXED_ARGUMENT_LIST = [
     label: 'image',
     value: {
       type: 'image',
-      originalContentUrl: `${location.href}assets/stp_image.png`,
-      previewImageUrl: `${location.href}assets/stp_image.png`,
+      originalContentUrl: `${base}/assets/stp_image.png`,
+      previewImageUrl: `${base}/assets/stp_image.png`,
     },
   },
   {
     label: 'video',
     value: {
       type: 'video',
-      originalContentUrl: `${location.href}assets/stp_video.mp4`,
-      previewImageUrl: `${location.href}assets/stp_video.mp4`,
+      originalContentUrl: `${base}/assets/stp_video.mp4`,
+      previewImageUrl: `${base}/assets/stp_video.mp4`,
       trackingId: 'track-id',
     },
   },
   {
     label: 'audio',
     value: {
       type: 'audio',
-      originalContentUrl: `${location.href}assets/stp_audio.mp3`,
+      originalContentUrl: `${base}/assets/stp_audio.mp3`,
       duration: 60000,
     },
   },