Skip to content

feat: recruiter facing notifications#3378

Merged
rebelchris merged 12 commits intomainfrom
feat-recruiter-messaging
Dec 30, 2025
Merged

feat: recruiter facing notifications#3378
rebelchris merged 12 commits intomainfrom
feat-recruiter-messaging

Conversation

@rebelchris
Copy link
Copy Markdown
Contributor

Adds two new recruiter facing messaging

  • new match for opportunity (candidate_accepted)
  • match goes live (after review)

TODO:

  • email parts

@pulumi
Copy link
Copy Markdown

pulumi Bot commented Dec 29, 2025

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

Resource Changes

    Name                                                   Type                                  Operation
+   api-sub-api.recruiter-new-candidate-notification       gcp:pubsub/subscription:Subscription  create
~   vpc-native-generate-search-invites-cron                kubernetes:batch/v1:CronJob           update
~   vpc-native-update-source-public-threshold-cron         kubernetes:batch/v1:CronJob           update
+   api-sub-api.recruiter-opportunity-live-notification    gcp:pubsub/subscription:Subscription  create
~   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-api-db-migration-e6617524                   kubernetes:batch/v1:Job               create
~   vpc-native-post-analytics-clickhouse-cron              kubernetes:batch/v1:CronJob           update
~   vpc-native-personalized-digest-deployment              kubernetes:apps/v1:Deployment         update
~   vpc-native-clean-zombie-opportunities-cron             kubernetes:batch/v1:CronJob           update
~   vpc-native-private-deployment                          kubernetes:apps/v1:Deployment         update
~   vpc-native-clean-stale-user-transactions-cron          kubernetes:batch/v1:CronJob           update
~   vpc-native-update-views-cron                           kubernetes:batch/v1:CronJob           update
~   vpc-native-temporal-deployment                         kubernetes:apps/v1:Deployment         update
~   vpc-native-hourly-notification-cron                    kubernetes:batch/v1:CronJob           update
~   vpc-native-update-current-streak-cron                  kubernetes:batch/v1:CronJob           update
~   vpc-native-user-profile-updated-sync-cron              kubernetes:batch/v1:CronJob           update
-   vpc-native-api-db-migration-195f8253                   kubernetes:batch/v1:Job               delete
+   vpc-native-api-clickhouse-migration-e6617524           kubernetes:batch/v1:Job               create
~   vpc-native-clean-zombie-users-cron                     kubernetes:batch/v1:CronJob           update
~   vpc-native-generic-referral-reminder-cron              kubernetes:batch/v1:CronJob           update
~   vpc-native-daily-digest-cron                           kubernetes:batch/v1:CronJob           update
-   vpc-native-api-clickhouse-migration-195f8253           kubernetes:batch/v1:Job               delete
~   vpc-native-clean-zombie-images-cron                    kubernetes:batch/v1:CronJob           update
~   vpc-native-update-highlighted-views-cron               kubernetes:batch/v1:CronJob           update
~   vpc-native-ws-deployment                               kubernetes:apps/v1:Deployment         update
~   vpc-native-validate-active-users-cron                  kubernetes:batch/v1:CronJob           update
~   vpc-native-bg-deployment                               kubernetes:apps/v1:Deployment         update
~   vpc-native-sync-subscription-with-cio-cron             kubernetes:batch/v1:CronJob           update
~   vpc-native-update-tags-str-cron                        kubernetes:batch/v1:CronJob           update
~   vpc-native-update-trending-cron                        kubernetes:batch/v1:CronJob           update
~   vpc-native-personalized-digest-cron                    kubernetes:batch/v1:CronJob           update
~   vpc-native-check-analytics-report-cron                 kubernetes:batch/v1:CronJob           update
~   vpc-native-deployment                                  kubernetes:apps/v1:Deployment         update
~   vpc-native-clean-gifted-plus-cron                      kubernetes:batch/v1:CronJob           update
~   vpc-native-post-analytics-history-day-clickhouse-cron  kubernetes:batch/v1:CronJob           update
~   vpc-native-update-tag-recommendations-cron             kubernetes:batch/v1:CronJob           update
~   vpc-native-clean-zombie-user-companies-cron            kubernetes:batch/v1:CronJob           update

Comment thread src/workers/notifications/recruiterNewCandidateNotification.ts
Comment thread src/workers/notifications/recruiterNewCandidateNotification.ts Outdated
const [candidate, opportunity, match, keywords] = await Promise.all([
con.getRepository(User).findOne({
where: { id: userId },
select: ['id', 'name', 'username'],
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

i think you need also the image here since we use it as an avatar for the notification

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't see you added the image here. Have you validated that?

Comment thread src/workers/newNotificationV2Mail.ts Outdated
Comment thread src/workers/newNotificationV2Mail.ts Outdated
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.

Nothing blocking, mostly best practices

@rebelchris rebelchris merged commit 50ec685 into main Dec 30, 2025
7 of 9 checks passed
@rebelchris rebelchris deleted the feat-recruiter-messaging branch December 30, 2025 11:53
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