Skip to content

refactor: remove deprecated community picks granted notification#3367

Merged
idoshamun merged 4 commits intomainfrom
remove-community-picks-granted
Dec 22, 2025
Merged

refactor: remove deprecated community picks granted notification#3367
idoshamun merged 4 commits intomainfrom
remove-community-picks-granted

Conversation

@idoshamun
Copy link
Copy Markdown
Member

No description provided.

Comment thread src/entity/user/UserState.ts Outdated
CommunityLinkAccess = 'community_link_access',
}

/**
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@capJavert i wasn't sure if we want to delete this entity now or just keep it as is for now. your call

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see we use it on anything, and we have other ways to save additional user info, lets remove!

@pulumi
Copy link
Copy Markdown

pulumi Bot commented Dec 21, 2025

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

Resource Changes

    Name                                                   Type                                  Operation
~   vpc-native-deployment                                  kubernetes:apps/v1:Deployment         update
~   vpc-native-clean-zombie-opportunities-cron             kubernetes:batch/v1:CronJob           update
~   vpc-native-update-source-tag-view-cron                 kubernetes:batch/v1:CronJob           update
~   vpc-native-update-trending-cron                        kubernetes:batch/v1:CronJob           update
~   vpc-native-validate-active-users-cron                  kubernetes:batch/v1:CronJob           update
~   vpc-native-temporal-deployment                         kubernetes:apps/v1:Deployment         update
~   vpc-native-sync-subscription-with-cio-cron             kubernetes:batch/v1:CronJob           update
~   vpc-native-bg-deployment                               kubernetes:apps/v1:Deployment         update
~   vpc-native-update-highlighted-views-cron               kubernetes:batch/v1:CronJob           update
-   vpc-native-api-clickhouse-migration-7a931086           kubernetes:batch/v1:Job               delete
~   vpc-native-post-analytics-clickhouse-cron              kubernetes:batch/v1:CronJob           update
+   vpc-native-api-clickhouse-migration-76efa219           kubernetes:batch/v1:Job               create
+   vpc-native-api-db-migration-76efa219                   kubernetes:batch/v1:Job               create
~   vpc-native-update-current-streak-cron                  kubernetes:batch/v1:CronJob           update
~   vpc-native-clean-zombie-users-cron                     kubernetes:batch/v1:CronJob           update
~   vpc-native-private-deployment                          kubernetes:apps/v1:Deployment         update
~   vpc-native-personalized-digest-cron                    kubernetes:batch/v1:CronJob           update
~   vpc-native-update-views-cron                           kubernetes:batch/v1:CronJob           update
~   vpc-native-update-source-public-threshold-cron         kubernetes:batch/v1:CronJob           update
~   vpc-native-hourly-notification-cron                    kubernetes:batch/v1:CronJob           update
~   vpc-native-check-analytics-report-cron                 kubernetes:batch/v1:CronJob           update
~   vpc-native-clean-zombie-images-cron                    kubernetes:batch/v1:CronJob           update
~   vpc-native-generate-search-invites-cron                kubernetes:batch/v1:CronJob           update
~   vpc-native-post-analytics-history-day-clickhouse-cron  kubernetes:batch/v1:CronJob           update
~   vpc-native-update-tags-str-cron                        kubernetes:batch/v1:CronJob           update
~   vpc-native-update-tag-recommendations-cron             kubernetes:batch/v1:CronJob           update
~   vpc-native-clean-stale-user-transactions-cron          kubernetes:batch/v1:CronJob           update
-   vpc-native-api-db-migration-7a931086                   kubernetes:batch/v1:Job               delete
~   vpc-native-daily-digest-cron                           kubernetes:batch/v1:CronJob           update
~   vpc-native-clean-gifted-plus-cron                      kubernetes:batch/v1:CronJob           update
~   vpc-native-ws-deployment                               kubernetes:apps/v1:Deployment         update
~   vpc-native-personalized-digest-deployment              kubernetes:apps/v1:Deployment         update
~   vpc-native-generic-referral-reminder-cron              kubernetes:batch/v1:CronJob           update
~   vpc-native-calculate-top-readers-cron                  kubernetes:batch/v1:CronJob           update
~   vpc-native-user-profile-updated-sync-cron              kubernetes:batch/v1:CronJob           update
-   api-sub-api.community-picks-granted-notification       gcp:pubsub/subscription:Subscription  delete
~   vpc-native-clean-zombie-user-companies-cron            kubernetes:batch/v1:CronJob           update

Comment thread .infra/common.ts
subscription: 'api.community-picks-failed-notification',
},
{
topic: 'community-link-access',
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also kill topic on streams?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes but only once i merge this

@idoshamun idoshamun enabled auto-merge (squash) December 22, 2025 14:16
@idoshamun idoshamun merged commit 1c60371 into main Dec 22, 2025
10 checks passed
@idoshamun idoshamun deleted the remove-community-picks-granted branch December 22, 2025 14:19
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.

3 participants