Skip to content

feat: Enterprise contact form#15941

Merged
corwintines merged 20 commits into
stagingfrom
enterprise-form
Aug 7, 2025
Merged

feat: Enterprise contact form#15941
corwintines merged 20 commits into
stagingfrom
enterprise-form

Conversation

@wackerow
Copy link
Copy Markdown
Member

@wackerow wackerow commented Jul 26, 2025

Description

  • Adds email contact form to Enterprise footer
  • Form forwards minimum viable information as email to Enterprise team using AWS SES

Preview link

https://deploy-preview-15941--ethereumorg.netlify.app/en/enterprise/#get-in-touch

@netlify
Copy link
Copy Markdown

netlify Bot commented Jul 26, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit f8b4747
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/6894e4623a39640008ad13da
😎 Deploy Preview https://deploy-preview-15941--ethereumorg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 56 (🟢 up 4 from production)
Accessibility: 94 (no change from production)
Best Practices: 92 (no change from production)
SEO: 99 (no change from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot added content 🖋️ This involves copy additions or edits translation 🌍 This is related to our Translation Program labels Jul 26, 2025
@wackerow wackerow marked this pull request as draft July 30, 2025 17:40
@github-actions github-actions Bot added the dependencies 📦 Changes related to project dependencies label Aug 5, 2025
@wackerow wackerow force-pushed the enterprise-form branch 2 times, most recently from efb8004 to c8e4ff7 Compare August 6, 2025 18:38
@wackerow wackerow changed the base branch from dev to master August 6, 2025 23:21
@wackerow wackerow marked this pull request as ready for review August 6, 2025 23:29
Comment thread package.json
Comment thread app/[locale]/enterprise/_components/ContactForm/index.tsx Outdated
Comment thread app/[locale]/enterprise/_components/ContactForm/index.tsx Outdated
Comment thread app/api/enterprise-contact/route.ts Outdated
@github-actions github-actions Bot added the tooling 🔧 Changes related to tooling of the project label Aug 7, 2025
Comment thread src/lib/utils/sanitize.ts
* @param input - The string to sanitize
* @returns Sanitized string with dangerous content removed
*/
export function sanitizeInput(input: string): string {
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@pettinarip per discussion, looks like we could need to install more dependencies that could be overkill for what we're doing here... as a fallback, asked claude to catch the main attack vectors.

I see this as a solution for the time, but to be investigated further as the repo advances.

Pulled it into lib/utils and removed the duplication.

pettinarip
pettinarip previously approved these changes Aug 7, 2025
Copy link
Copy Markdown
Member

@pettinarip pettinarip left a comment

Choose a reason for hiding this comment

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

LGTM

message: sanitizeInput(formData.message),
}

const response = await fetch("/api/enterprise-contact", {
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.

good place to use react-query with useMutation

@wackerow wackerow changed the base branch from master to staging August 7, 2025 18:35
@wackerow wackerow dismissed pettinarip’s stale review August 7, 2025 18:35

The base branch was changed.

@corwintines corwintines merged commit aa52a69 into staging Aug 7, 2025
18 of 22 checks passed
@corwintines corwintines deleted the enterprise-form branch August 7, 2025 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content 🖋️ This involves copy additions or edits dependencies 📦 Changes related to project dependencies tooling 🔧 Changes related to tooling of the project translation 🌍 This is related to our Translation Program

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants