CS-60 [Improvement] [Feature] - Send onboarding email for new employee#2102
CS-60 [Improvement] [Feature] - Send onboarding email for new employee#2102github-actions[bot] wants to merge 4 commits intomainfrom
Conversation
PR SummaryMedium Risk Overview The invite UI ( Updates Written by Cursor Bugbot for commit 602c001. This will update automatically on new commits. Configure here. |
apps/app/src/app/(app)/[orgId]/people/all/actions/addEmployeeWithoutInvite.ts
Show resolved
Hide resolved
apps/app/src/app/(app)/[orgId]/people/all/actions/addEmployeeWithoutInvite.ts
Outdated
Show resolved
Hide resolved
apps/app/src/app/(app)/[orgId]/people/all/actions/addEmployeeWithoutInvite.ts
Outdated
Show resolved
Hide resolved
apps/app/src/app/(app)/[orgId]/people/all/actions/addEmployeeWithoutInvite.ts
Outdated
Show resolved
Hide resolved
…es inconsistent state
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
|
|
||
| const domain = isProdEnv ? 'app.trycomp.ai' : 'localhost:3002'; | ||
| const inviteLink = `${protocol}://${domain}/${organizationId}`; | ||
|
|
There was a problem hiding this comment.
Invite link uses wrong domain
High Severity
inviteLink is built from a hardcoded domain (app.trycomp.ai or localhost:3002) based on NEXT_PUBLIC_PORTAL_URL string matching, instead of using the actual configured portal base URL. This can send employees/contractors to the admin app (where they may be blocked) or to localhost in non-prod environments.


This is an automated pull request to merge chas/send-onboarding-email-to-employee into dev.
It was created by the [Auto Pull Request] action.