Skip to content

feat(api): email org owners/admins when a portal access request is submitted (CS-522) - #3494

Merged
tofikwest merged 4 commits into
mainfrom
chas/send-email-for-access-requests
Jul 27, 2026
Merged

feat(api): email org owners/admins when a portal access request is submitted (CS-522)#3494
tofikwest merged 4 commits into
mainfrom
chas/send-email-for-access-requests

Conversation

@github-actions

@github-actions github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

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


Summary by cubic

Emails org owners/admins when an access-request form is submitted, including a direct “Review Request” link. Implements CS-522 and respects user email preferences.

  • New Features

    • Added EvidenceFormsNotifierService to notify active owners/admins, skipping the submitter and users unsubscribed via @trycompai/email (also excludes deactivated/inactive members).
    • Introduced EvidenceAccessRequestSubmittedEmail with accounts, permissions, reason, a “Review Request” button, and an unsubscribe link.
    • Hooked into EvidenceFormsService for access-request submissions only; registered via EvidenceFormsModule. Review URL built from NEXT_PUBLIC_APP_URL or BETTER_AUTH_URL (fallback https://app.trycomp.ai). Tests cover recipients, unsubscribe behavior, and non-triggering for other form types.
  • Bug Fixes

    • Updated notifier spec to match the isActive/deactivated recipient filter.

Written for commit f497d35. Summary will update on new commits.

Review in cubic

@vercel

vercel Bot commented Jul 23, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
app Ready Ready Preview, Comment Jul 27, 2026 4:57pm
comp-framework-editor Ready Ready Preview, Comment Jul 27, 2026 4:57pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
portal Skipped Skipped Jul 27, 2026 4:57pm

Request Review

@chasprowebdev chasprowebdev changed the title [dev] [chasprowebdev] chas/send-email-for-access-requests feat(api): email org owners/admins when a portal access request is submitted (CS-522) Jul 23, 2026
@linear

linear Bot commented Jul 23, 2026

Copy link
Copy Markdown

CS-522

@cubic-dev-ai cubic-dev-ai Bot left a comment

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.

All reported issues were addressed across 6 files

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

Comment thread apps/api/src/evidence-forms/evidence-forms-notifier.service.ts
Comment thread apps/api/src/evidence-forms/evidence-forms-notifier.service.ts
@vercel
vercel Bot temporarily deployed to Preview – app July 23, 2026 21:11 Inactive
@vercel
vercel Bot temporarily deployed to Preview – portal July 23, 2026 21:11 Inactive
@chasprowebdev

Copy link
Copy Markdown
Contributor

@cubic-dev-ai please review it.

@cubic-dev-ai

cubic-dev-ai Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai please review it.

@chasprowebdev I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment

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.

No issues found across 6 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@chasprowebdev chasprowebdev self-assigned this Jul 23, 2026
@vercel
vercel Bot temporarily deployed to Preview – portal July 27, 2026 16:53 Inactive
@tofikwest
tofikwest merged commit 258ce45 into main Jul 27, 2026
10 of 11 checks passed
@tofikwest
tofikwest deleted the chas/send-email-for-access-requests branch July 27, 2026 16:56
claudfuen pushed a commit that referenced this pull request Jul 27, 2026
# [3.110.0](v3.109.0...v3.110.0) (2026-07-27)

### Bug Fixes

* **app:** make invite modal manual rows scrollable so action buttons stay visible ([#3510](#3510)) ([9926408](9926408))
* **audit:** stop logging read endpoints as mutations + name task-item events ([#3508](#3508)) ([3f9e1cf](3f9e1cf))
* **audit:** stop plaintext secrets leaking into the audit log ([#3512](#3512)) ([55a48fb](55a48fb))

### Features

* **api:** email org owners/admins when a portal access request is submitted (CS-522) ([#3494](#3494)) ([258ce45](258ce45))
@claudfuen

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 3.110.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants