Skip to content

feat: claim anon opportunities#3316

Merged
capJavert merged 4 commits intomainfrom
claim-anon-opportunities
Dec 2, 2025
Merged

feat: claim anon opportunities#3316
capJavert merged 4 commits intomainfrom
claim-anon-opportunities

Conversation

@capJavert
Copy link
Copy Markdown
Contributor

@capJavert capJavert commented Dec 1, 2025

When user registers while having anon opportunities waiting from self serve flow we register him as a recruiter for that opportunity.

Also added logic to create organization by name if not set on opportunity already.

@capJavert capJavert self-assigned this Dec 1, 2025
@capJavert capJavert requested a review from a team as a code owner December 1, 2025 13:30
@pulumi
Copy link
Copy Markdown

pulumi Bot commented Dec 1, 2025

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

Resource Changes

    Name                                                   Type                           Operation
~   vpc-native-generate-search-invites-cron                kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-zombie-user-companies-cron            kubernetes:batch/v1:CronJob    update
~   vpc-native-update-views-cron                           kubernetes:batch/v1:CronJob    update
~   vpc-native-update-tag-recommendations-cron             kubernetes:batch/v1:CronJob    update
~   vpc-native-update-tags-str-cron                        kubernetes:batch/v1:CronJob    update
+   vpc-native-api-clickhouse-migration-173515f4           kubernetes:batch/v1:Job        create
~   vpc-native-daily-digest-cron                           kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-gifted-plus-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-clean-zombie-users-cron                     kubernetes:batch/v1:CronJob    update
~   vpc-native-sync-subscription-with-cio-cron             kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-stale-user-transactions-cron          kubernetes:batch/v1:CronJob    update
~   vpc-native-validate-active-users-cron                  kubernetes:batch/v1:CronJob    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-personalized-digest-cron                    kubernetes:batch/v1:CronJob    update
~   vpc-native-generic-referral-reminder-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-post-analytics-clickhouse-cron              kubernetes:batch/v1:CronJob    update
~   vpc-native-personalized-digest-deployment              kubernetes:apps/v1:Deployment  update
~   vpc-native-deployment                                  kubernetes:apps/v1:Deployment  update
~   vpc-native-check-analytics-report-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-update-trending-cron                        kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-zombie-images-cron                    kubernetes:batch/v1:CronJob    update
~   vpc-native-bg-deployment                               kubernetes:apps/v1:Deployment  update
+   vpc-native-api-db-migration-173515f4                   kubernetes:batch/v1:Job        create
~   vpc-native-temporal-deployment                         kubernetes:apps/v1:Deployment  update
~   vpc-native-clean-zombie-opportunities-cron             kubernetes:batch/v1:CronJob    update
-   vpc-native-api-db-migration-ba1273a5                   kubernetes:batch/v1:Job        delete
~   vpc-native-private-deployment                          kubernetes:apps/v1:Deployment  update
-   vpc-native-api-clickhouse-migration-ba1273a5           kubernetes:batch/v1:Job        delete

Comment thread src/routes/private.ts
Comment on lines +67 to +76
if (opportunities.length > 0) {
logger.info(
{
anonUserId: body.id,
userId: operationResult.userId,
opportunities,
},
'Claimed anon opportunities for new user',
);
}
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Will remove, added so I can monitor for a bit

updatedAt: Date;

@Column({ type: 'text' })
@Index('IDX_organization_name_unique', { unique: true })
Copy link
Copy Markdown
Contributor Author

@capJavert capJavert Dec 1, 2025

Choose a reason for hiding this comment

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

Added so that organization name is unique to avoid people adding well known organizations that were already created.

Still anyone can add any organization so we need to be careful and monitor what gets added. Since we will be doing some manual approval before self serve goes live I am counting on that.

Copy link
Copy Markdown
Contributor

@rebelchris rebelchris left a comment

Choose a reason for hiding this comment

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

looks good to me

@capJavert capJavert merged commit dc4cdd3 into main Dec 2, 2025
9 checks passed
@capJavert capJavert deleted the claim-anon-opportunities branch December 2, 2025 10:09
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