From 86d264113d4195c5cd11dec0f58891bcd487ec48 Mon Sep 17 00:00:00 2001
From: canghai118 <3394863+canghai118@users.noreply.github.com>
Date: Thu, 21 May 2026 09:43:00 +0800
Subject: [PATCH 1/3] feat(post): share request button copies canonical URL via
 toast

Use vue-sonner for feedback and move global Toaster to top-center.
Share URL is built from the post slug so it stays correct when the
post is opened in a modal over a different page.
---
 app/app.vue                        |  2 +-
 app/components/post/PostDetail.vue | 14 +++++++++++++-
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/app/app.vue b/app/app.vue
index 898cc04..0efe9cc 100644
--- a/app/app.vue
+++ b/app/app.vue
@@ -9,5 +9,5 @@ import 'vue-sonner/style.css'
     <NuxtPage />
   </NuxtLayout>
   <ConfirmDialogProvider />
-  <Toaster position="bottom-right" :duration="3000" rich-colors close-button />
+  <Toaster position="top-center" :duration="3000" rich-colors close-button />
 </template>
diff --git a/app/components/post/PostDetail.vue b/app/components/post/PostDetail.vue
index 00884ec..10ad59e 100644
--- a/app/components/post/PostDetail.vue
+++ b/app/components/post/PostDetail.vue
@@ -2,6 +2,7 @@
 import { MdEditor } from 'md-editor-v3'
 import 'md-editor-v3/lib/style.css'
 import '~/assets/css/md-editor-preview.css'
+import { toast } from 'vue-sonner'
 import { sanitizeAttachmentHtml } from '~/utils/attachment';
 
 
@@ -280,6 +281,17 @@ async function handleDeleteComment(commentId: string) {
 watch(commentSort, () => {
   store.fetchComments(props.slug, commentSort.value)
 })
+
+async function handleShare() {
+  // Canonical post URL — independent of current location (post may be opened in a modal)
+  const url = `${window.location.origin}/p/${props.slug}`
+  try {
+    await navigator.clipboard.writeText(url)
+    toast.success('Link copied to clipboard')
+  } catch {
+    toast.error('Failed to copy link')
+  }
+}
 </script>
 
 <template>
@@ -475,7 +487,7 @@ watch(commentSort, () => {
         </div>
         <div class="pt-2 flex flex-col gap-2">
           <Button variant="secondary" :disabled="isMerged" :class="isMerged ? 'opacity-50 cursor-not-allowed' : ''"><Icon name="lucide:bell" size="18" /> Subscribe to Updates</Button>
-          <Button variant="outline" class="text-primary" :disabled="isMerged" :class="isMerged ? 'opacity-50 cursor-not-allowed' : ''"><Icon name="lucide:share-2" size="18" /> Share Request</Button>
+          <Button variant="outline" class="text-primary" :disabled="isMerged" :class="isMerged ? 'opacity-50 cursor-not-allowed' : ''" @click="handleShare"><Icon name="lucide:share-2" size="18" /> Share Request</Button>
         </div>
       </div>
       <SimilarPostsPanel v-if="post.id && !isMerged" :post-id="post.id" :is-admin="isOrgManager" @merge="handleSimilarMerge" />

From e42637f70069d61e9355a99df04cd2e0f047e78e Mon Sep 17 00:00:00 2001
From: canghai118 <3394863+canghai118@users.noreply.github.com>
Date: Thu, 21 May 2026 09:46:33 +0800
Subject: [PATCH 2/3] chore(post): hide subscribe button until notification
 backend lands

---
 app/components/post/PostDetail.vue | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/app/components/post/PostDetail.vue b/app/components/post/PostDetail.vue
index 10ad59e..d9f4fdb 100644
--- a/app/components/post/PostDetail.vue
+++ b/app/components/post/PostDetail.vue
@@ -486,7 +486,9 @@ async function handleShare() {
           </div>
         </div>
         <div class="pt-2 flex flex-col gap-2">
+          <!-- Subscribe to Updates — hidden until the notification backend lands; unhide when ready
           <Button variant="secondary" :disabled="isMerged" :class="isMerged ? 'opacity-50 cursor-not-allowed' : ''"><Icon name="lucide:bell" size="18" /> Subscribe to Updates</Button>
+          -->
           <Button variant="outline" class="text-primary" :disabled="isMerged" :class="isMerged ? 'opacity-50 cursor-not-allowed' : ''" @click="handleShare"><Icon name="lucide:share-2" size="18" /> Share Request</Button>
         </div>
       </div>

From fc71dcd14d4f6488ebbcca6997e440e10f2f60d9 Mon Sep 17 00:00:00 2001
From: canghai118 <3394863+canghai118@users.noreply.github.com>
Date: Thu, 21 May 2026 09:50:41 +0800
Subject: [PATCH 3/3] fix(changelog): prompt login on reaction click for
 anonymous users

Reactions previously called the API immediately, which 401'd for
anonymous users and left a brief optimistic highlight that rolled
back with no explanation. Now toggle() opens the global LoginModal
before any API call, matching the post/comment login-guard pattern.
---
 app/components/changelog/ChangelogReactions.vue | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/app/components/changelog/ChangelogReactions.vue b/app/components/changelog/ChangelogReactions.vue
index 1231540..ba9f728 100644
--- a/app/components/changelog/ChangelogReactions.vue
+++ b/app/components/changelog/ChangelogReactions.vue
@@ -15,6 +15,10 @@ const picked = reactive<Record<string, boolean>>({})
 const pickerOpen = ref(false)
 const rootRef = ref<HTMLElement | null>(null)
 
+const { data: session } = useAuthSession()
+const isLoggedIn = computed(() => !!session.value?.user)
+const loginModal = useLoginModal()
+
 // Initialize picked state from userReactions
 for (const emoji of props.userReactions) {
   picked[emoji] = true
@@ -29,6 +33,11 @@ const displayedReactions = computed(() =>
 )
 
 async function toggle(emoji: string) {
+  if (!isLoggedIn.value) {
+    pickerOpen.value = false
+    return loginModal.open()
+  }
+
   const wasActive = picked[emoji]
 
   // Optimistic update