Skip to content

fix: use via instead of ref query param on redirect - #4025

Merged
idoshamun merged 2 commits into
mainfrom
eng-1897-feedback-bug-report-links-to-substack-are-broken-due-to
Jul 27, 2026
Merged

fix: use via instead of ref query param on redirect#4025
idoshamun merged 2 commits into
mainfrom
eng-1897-feedback-bug-report-links-to-substack-are-broken-due-to

Conversation

@idoshamun

@idoshamun idoshamun commented Jul 27, 2026

Copy link
Copy Markdown
Member

Links to Substack (and some other sites) break when we append ref as a query param on the outbound redirect. Renamed the attribution param to via, the conventional share-attribution name.

Only affects /r/:postId redirects — the roadmap.sh?ref=dailydev partner link is unchanged.

https://claude.ai/code/session_013o9WhBTGKXCzrUzmUCVTZE

Substack (and some other sites) break on an appended `ref` query param,
so use `r` for the outbound redirect attribution instead.

Claude-Session: https://claude.ai/code/session_013o9WhBTGKXCzrUzmUCVTZE
@pulumi

pulumi Bot commented Jul 27, 2026

Copy link
Copy Markdown

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

Resource Changes

    Name                                                       Type                           Operation
~   vpc-native-daily-digest-cron                               kubernetes:batch/v1:CronJob    update
~   vpc-native-bg-deployment                                   kubernetes:apps/v1:Deployment  update
~   vpc-native-clean-zombie-opportunities-cron                 kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-expired-better-auth-sessions-cron         kubernetes:batch/v1:CronJob    update
~   vpc-native-validate-active-users-cron                      kubernetes:batch/v1:CronJob    update
~   vpc-native-private-deployment                              kubernetes:apps/v1:Deployment  update
~   vpc-native-post-analytics-history-day-clickhouse-cron      kubernetes:batch/v1:CronJob    update
-   vpc-native-api-clickhouse-migration-feea5bf5               kubernetes:batch/v1:Job        delete
~   vpc-native-clean-gifted-plus-cron                          kubernetes:batch/v1:CronJob    update
~   vpc-native-update-current-streak-cron                      kubernetes:batch/v1:CronJob    update
~   vpc-native-materialize-monthly-best-post-archives-cron     kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-old-notifications-cron                    kubernetes:batch/v1:CronJob    update
~   vpc-native-expire-super-agent-trial-cron                   kubernetes:batch/v1:CronJob    update
~   vpc-native-interest-scheduled-run-cron                     kubernetes:batch/v1:CronJob    update
~   vpc-native-hourly-notification-cron                        kubernetes:batch/v1:CronJob    update
~   vpc-native-user-profile-updated-sync-cron                  kubernetes:batch/v1:CronJob    update
~   vpc-native-user-profile-analytics-clickhouse-cron          kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-zombie-images-cron                        kubernetes:batch/v1:CronJob    update
~   vpc-native-update-achievement-rarity-cron                  kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-stale-user-transactions-cron              kubernetes:batch/v1:CronJob    update
~   vpc-native-generic-referral-reminder-cron                  kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-channel-highlights-cron                   kubernetes:batch/v1:CronJob    update
~   vpc-native-update-tags-str-cron                            kubernetes:batch/v1:CronJob    update
~   vpc-native-materialize-yearly-best-post-archives-cron      kubernetes:batch/v1:CronJob    update
~   vpc-native-channel-highlights-cron                         kubernetes:batch/v1:CronJob    update
~   vpc-native-check-analytics-report-cron                     kubernetes:batch/v1:CronJob    update
~   vpc-native-ws-deployment                                   kubernetes:apps/v1:Deployment  update
~   vpc-native-rotate-weekly-quests-cron                       kubernetes:batch/v1:CronJob    update
~   vpc-native-update-trending-cron                            kubernetes:batch/v1:CronJob    update
~   vpc-native-generate-search-invites-cron                    kubernetes:batch/v1:CronJob    update
~   vpc-native-personalized-digest-deployment                  kubernetes:apps/v1:Deployment  update
~   vpc-native-worker-job-deployment                           kubernetes:apps/v1:Deployment  update
~   vpc-native-deployment                                      kubernetes:apps/v1:Deployment  update
~   vpc-native-temporal-deployment                             kubernetes:apps/v1:Deployment  update
~   vpc-native-clean-zombie-users-cron                         kubernetes:batch/v1:CronJob    update
~   vpc-native-personalized-digest-cron                        kubernetes:batch/v1:CronJob    update
~   vpc-native-channel-digests-cron                            kubernetes:batch/v1:CronJob    update
~   vpc-native-user-posts-analytics-refresh-cron               kubernetes:batch/v1:CronJob    update
~   vpc-native-user-profile-analytics-history-clickhouse-cron  kubernetes:batch/v1:CronJob    update
~   vpc-native-post-analytics-clickhouse-cron                  kubernetes:batch/v1:CronJob    update
-   vpc-native-api-db-migration-feea5bf5                       kubernetes:batch/v1:Job        delete
... and 16 other changes

@idoshamun idoshamun changed the title fix: use r instead of ref query param on redirect fix: use via instead of ref query param on redirect Jul 27, 2026
@idoshamun
idoshamun merged commit 118bd95 into main Jul 27, 2026
9 checks passed
@idoshamun
idoshamun deleted the eng-1897-feedback-bug-report-links-to-substack-are-broken-due-to branch July 27, 2026 06:34
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