Skip to content

fix: slack for internal review#3411

Merged
rebelchris merged 8 commits intomainfrom
feat-internal-review
Jan 8, 2026
Merged

fix: slack for internal review#3411
rebelchris merged 8 commits intomainfrom
feat-internal-review

Conversation

@rebelchris
Copy link
Copy Markdown
Contributor

Adds new intermediate state of internal review.
It should send a interactive slack message which we can review.

If fails we can always manually do database work :)

@rebelchris rebelchris requested a review from idoshamun January 8, 2026 09:31
@rebelchris rebelchris requested review from a team and capJavert as code owners January 8, 2026 09:31
@pulumi
Copy link
Copy Markdown

pulumi Bot commented Jan 8, 2026

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

Resource Changes

    Name                                                   Type                                  Operation
-   vpc-native-api-db-migration-15259f48                   kubernetes:batch/v1:Job               delete
~   vpc-native-clean-zombie-user-companies-cron            kubernetes:batch/v1:CronJob           update
-   vpc-native-api-clickhouse-migration-15259f48           kubernetes:batch/v1:Job               delete
~   vpc-native-clean-zombie-users-cron                     kubernetes:batch/v1:CronJob           update
~   vpc-native-validate-active-users-cron                  kubernetes:batch/v1:CronJob           update
+   api-sub-api.candidate-review-opportunity-slack         gcp:pubsub/subscription:Subscription  create
~   vpc-native-clean-zombie-images-cron                    kubernetes:batch/v1:CronJob           update
~   vpc-native-clean-gifted-plus-cron                      kubernetes:batch/v1:CronJob           update
~   vpc-native-update-views-cron                           kubernetes:batch/v1:CronJob           update
~   vpc-native-personalized-digest-cron                    kubernetes:batch/v1:CronJob           update
~   vpc-native-update-trending-cron                        kubernetes:batch/v1:CronJob           update
~   vpc-native-user-profile-updated-sync-cron              kubernetes:batch/v1:CronJob           update
~   vpc-native-update-source-public-threshold-cron         kubernetes:batch/v1:CronJob           update
~   vpc-native-sync-subscription-with-cio-cron             kubernetes:batch/v1:CronJob           update
~   vpc-native-deployment                                  kubernetes:apps/v1:Deployment         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-private-deployment                          kubernetes:apps/v1:Deployment         update
~   vpc-native-clean-zombie-opportunities-cron             kubernetes:batch/v1:CronJob           update
~   vpc-native-post-analytics-clickhouse-cron              kubernetes:batch/v1:CronJob           update
~   vpc-native-hourly-notification-cron                    kubernetes:batch/v1:CronJob           update
~   vpc-native-personalized-digest-deployment              kubernetes:apps/v1:Deployment         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-update-current-streak-cron                  kubernetes:batch/v1:CronJob           update
+   vpc-native-api-clickhouse-migration-bd8da737           kubernetes:batch/v1:Job               create
~   vpc-native-ws-deployment                               kubernetes:apps/v1:Deployment         update
~   vpc-native-update-tags-str-cron                        kubernetes:batch/v1:CronJob           update
~   vpc-native-bg-deployment                               kubernetes:apps/v1:Deployment         update
~   vpc-native-update-source-tag-view-cron                 kubernetes:batch/v1:CronJob           update
~   vpc-native-calculate-top-readers-cron                  kubernetes:batch/v1:CronJob           update
~   vpc-native-update-tag-recommendations-cron             kubernetes:batch/v1:CronJob           update
~   vpc-native-check-analytics-report-cron                 kubernetes:batch/v1:CronJob           update
~   vpc-native-temporal-deployment                         kubernetes:apps/v1:Deployment         update
~   vpc-native-daily-digest-cron                           kubernetes:batch/v1:CronJob           update
~   vpc-native-clean-stale-user-transactions-cron          kubernetes:batch/v1:CronJob           update
+   vpc-native-api-db-migration-bd8da737                   kubernetes:batch/v1:Job               create

Comment thread __tests__/schema/opportunity.ts
Comment thread src/common/opportunity/pubsub.ts Outdated
Comment thread src/common/typedPubsub.ts Outdated
Comment thread src/workers/candidateReviewOpportunitySlack.ts Outdated
@rebelchris rebelchris requested a review from capJavert January 8, 2026 10:51
Comment thread src/routes/integrations/slack.ts Outdated
Comment thread src/common/opportunity/pubsub.ts Outdated
Comment thread src/routes/integrations/slack.ts
Comment thread src/workers/candidateReviewOpportunitySlack.ts
Copy link
Copy Markdown
Member

@idoshamun idoshamun left a comment

Choose a reason for hiding this comment

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

One request for the slack message but looks great

Comment thread src/common/schema/slack.ts Outdated
Comment thread src/routes/integrations/slack.ts Outdated
@rebelchris rebelchris merged commit 20285c0 into main Jan 8, 2026
10 checks passed
@rebelchris rebelchris deleted the feat-internal-review branch January 8, 2026 13:27
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