Skip to content

Conversation

@rebelchris
Copy link
Contributor

Summary

  • Add api.v1.feedback-updated topic triggered when feedback is updated via CDC
  • Create feedbackUpdatedSlack worker that sends Slack messages to user-feedback channel for completed feedback
  • Messages include: user profile link, category, sentiment, urgency, summary, description, page URL, tags, and Linear issue link
  • Extract shared feedback helpers (getCategoryDisplayName, getSentimentEmoji) to src/common/feedback.ts to reduce duplication

Test plan

  • Added unit tests for feedbackUpdatedSlack worker
  • Tests verify: worker registration, sends message for completed feedback, skips non-completed, skips not found
  • Set SLACK_USER_FEEDBACK_WEBHOOK env var in production

🤖 Generated with Claude Code

- Add api.v1.feedback-updated topic triggered on feedback updates
- Create feedbackUpdatedSlack worker that sends Slack messages for completed feedback
- Include user profile, description, classification, page URL, and Linear link
- Extract shared feedback helpers (getCategoryDisplayName, getSentimentEmoji) to src/common/feedback.ts
- Add SLACK_USER_FEEDBACK_WEBHOOK env var

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@pulumi
Copy link

pulumi bot commented Jan 30, 2026

🍹 The Update (preview) for dailydotdev/api/prod (at d89f4dd) was successful.

Resource Changes

    Name                                                       Type                                  Operation
~   vpc-native-daily-digest-cron                               kubernetes:batch/v1:CronJob           update
~   vpc-native-temporal-deployment                             kubernetes:apps/v1:Deployment         update
~   vpc-native-check-analytics-report-cron                     kubernetes:batch/v1:CronJob           update
~   vpc-native-personalized-digest-deployment                  kubernetes:apps/v1:Deployment         update
~   vpc-native-update-source-tag-view-cron                     kubernetes:batch/v1:CronJob           update
~   vpc-native-update-views-cron                               kubernetes:batch/v1:CronJob           update
+   api-sub-api.feedback-updated-slack                         gcp:pubsub/subscription:Subscription  create
~   vpc-native-bg-deployment                                   kubernetes:apps/v1:Deployment         update
+-  vpc-native-k8s-secret                                      kubernetes:core/v1:Secret             create-replacement
~   vpc-native-update-tag-recommendations-cron                 kubernetes:batch/v1:CronJob           update
~   vpc-native-update-source-public-threshold-cron             kubernetes:batch/v1:CronJob           update
~   vpc-native-deployment                                      kubernetes:apps/v1:Deployment         update
-   vpc-native-api-clickhouse-migration-85ccd69f               kubernetes:batch/v1:Job               delete
+   vpc-native-api-db-migration-ea331613                       kubernetes:batch/v1:Job               create
~   vpc-native-hourly-notification-cron                        kubernetes:batch/v1:CronJob           update
~   vpc-native-post-analytics-clickhouse-cron                  kubernetes:batch/v1:CronJob           update
~   vpc-native-private-deployment                              kubernetes:apps/v1:Deployment         update
-   vpc-native-api-db-migration-85ccd69f                       kubernetes:batch/v1:Job               delete
~   vpc-native-user-profile-analytics-history-clickhouse-cron  kubernetes:batch/v1:CronJob           update
~   vpc-native-calculate-top-readers-cron                      kubernetes:batch/v1:CronJob           update
~   vpc-native-validate-active-users-cron                      kubernetes:batch/v1:CronJob           update
~   vpc-native-generic-referral-reminder-cron                  kubernetes:batch/v1:CronJob           update
~   vpc-native-post-analytics-history-day-clickhouse-cron      kubernetes:batch/v1:CronJob           update
~   vpc-native-sync-subscription-with-cio-cron                 kubernetes:batch/v1:CronJob           update
~   vpc-native-ws-deployment                                   kubernetes:apps/v1:Deployment         update
~   vpc-native-update-current-streak-cron                      kubernetes:batch/v1:CronJob           update
+   vpc-native-api-clickhouse-migration-ea331613               kubernetes:batch/v1:Job               create
~   vpc-native-update-tags-str-cron                            kubernetes:batch/v1:CronJob           update
~   vpc-native-clean-zombie-user-companies-cron                kubernetes:batch/v1:CronJob           update
~   vpc-native-update-trending-cron                            kubernetes:batch/v1:CronJob           update
~   vpc-native-clean-gifted-plus-cron                          kubernetes:batch/v1:CronJob           update
~   vpc-native-clean-zombie-opportunities-cron                 kubernetes:batch/v1:CronJob           update
~   vpc-native-expire-super-agent-trial-cron                   kubernetes:batch/v1:CronJob           update
~   vpc-native-user-profile-updated-sync-cron                  kubernetes:batch/v1:CronJob           update
~   vpc-native-generate-search-invites-cron                    kubernetes:batch/v1:CronJob           update
~   vpc-native-update-highlighted-views-cron                   kubernetes:batch/v1:CronJob           update
~   vpc-native-clean-stale-user-transactions-cron              kubernetes:batch/v1:CronJob           update
~   vpc-native-personalized-digest-cron                        kubernetes:batch/v1:CronJob           update
~   vpc-native-clean-zombie-images-cron                        kubernetes:batch/v1:CronJob           update
~   vpc-native-user-profile-analytics-clickhouse-cron          kubernetes:batch/v1:CronJob           update
~   vpc-native-clean-zombie-users-cron                         kubernetes:batch/v1:CronJob           update
... and 2 other changes

@rebelchris rebelchris merged commit 0d84a48 into main Jan 30, 2026
10 checks passed
@rebelchris rebelchris deleted the feat/feedback-updated-slack-notification branch January 30, 2026 10:58
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.

2 participants