Skip to content

feat: add "Say thanks" action to received Award notification#6302

Merged
capJavert merged 7 commits into
mainfrom
eng-1774-feedback-feature-request-user-suggests-adding-a-feature-to
Jul 15, 2026
Merged

feat: add "Say thanks" action to received Award notification#6302
capJavert merged 7 commits into
mainfrom
eng-1774-feedback-feature-request-user-suggests-adding-a-feature-to

Conversation

@capJavert

@capJavert capJavert commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

What

Adds a lightweight, one-tap "Say thanks" action so a recipient of a Cores Award can acknowledge the sender directly from the notifications page — modeled on the existing "Follow back" action on the UserFollow notification.

Changes

  • New NotificationSayThanksButton rendered inline on the UserReceivedAward notification item only, mirroring NotificationFollowUserButton (placement, -ml-3.5/mt-3 alignment, tertiary → primary state swap).
  • Calls the new sayThanksForAward(transactionId) GraphQL mutation and swaps to a disabled "Thanked" state on success. A CONFLICT response (already thanked elsewhere) resolves to the same "Thanked" state rather than an error toast.
  • Registers the new UserAwardThanked notification type in the shared notifications enum, theme, and category maps so the sender's acknowledgement renders correctly.

Key decisions

  • Reuse over new primitives: mirrors the "Follow back" pattern rather than building a messaging/reply UI.
  • No affordance in ListAwardsModal — the single entry point is the notification item.
  • Pragmatic post-action state: the notification payload doesn't carry thankedAt, so the button uses local state + the mutation's idempotency guard; the backend prevents any double-notify.

Backend counterpart (mutation, notification worker, thankedAt flag) ships in the daily-api PR.

Closes ENG-1774


Created by Huginn 🐦‍⬛

Preview domain

https://eng-1774-feedback-feature-reques.preview.app.daily.dev

Adds an inline "Say thanks" button on the UserReceivedAward notification
item, mirroring the "Follow back" action on UserFollow. Calls the new
sayThanksForAward mutation and swaps to a disabled "Thanked" state on
success. No affordance is added to ListAwardsModal.

Also renders the resulting UserAwardThanked notification for the sender.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented Jul 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
daily-webapp Error Error Jul 15, 2026 9:06am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
storybook Ignored Ignored Jul 15, 2026 9:06am

Request Review

@capJavert

Copy link
Copy Markdown
Contributor Author

@bobbyiliev this one is for you

@capJavert capJavert disabled auto-merge July 15, 2026 09:08
@capJavert capJavert merged commit 7890209 into main Jul 15, 2026
11 of 12 checks passed
@capJavert capJavert deleted the eng-1774-feedback-feature-request-user-suggests-adding-a-feature-to branch July 15, 2026 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant