Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add /contact page, use instead of email #298

Merged
merged 3 commits into from
Dec 26, 2024
Merged

Conversation

rfay
Copy link
Member

@rfay rfay commented Dec 24, 2024

The Issue

  • I noticed that the email in the footer didn't actually work, at least on my browser
  • The expected /contact page didn't exist, most org sites want that.
  • Add LinkedIn to footer socials
  • Minor improvement to layout of snailmail contact

Rendered at https://20241224-contact-and-linkedi.ddev-com-front-end.pages.dev/

See especially the footer and /contact pages

@rfay rfay marked this pull request as ready for review December 24, 2024 20:59
@@ -11,17 +12,18 @@ export const GITHUB_ISSUES_URL = "https://github.com/ddev/ddev/issues"
export const DOCUMENTATION_URL = "https://ddev.readthedocs.io/"
export const DISCORD_URL = "/s/discord"
export const EMAIL_URL = "mailto:support%40ddev.com"
Copy link
Collaborator

Choose a reason for hiding this comment

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

This typo to be specific we need to replace %40 for @

Copy link
Member Author

Choose a reason for hiding this comment

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

This works in regular links, and the idea is to prevent web crawlers from capturing the email address.

Copy link
Collaborator

Choose a reason for hiding this comment

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

oooh, that makes sense, I thought this was the original issue with the email on the footer.

@bmartinez287
Copy link
Collaborator

Screenshot 2024-12-25 at 9 12 54 PM the email issue seems to carry the on both places. It looks like it was a typo on that variable. i added a comment to that.

@bmartinez287
Copy link
Collaborator

other than that the contact page seems fine.

Copy link

cloudflare-workers-and-pages bot commented Dec 26, 2024

Deploying ddev-com-front-end with  Cloudflare Pages  Cloudflare Pages

Latest commit: 75972f6
Status: ✅  Deploy successful!
Preview URL: https://52f7a161.ddev-com-front-end.pages.dev
Branch Preview URL: https://20241224-contact-and-linkedi.ddev-com-front-end.pages.dev

View logs

Copy link
Member

@stasadev stasadev left a comment

Choose a reason for hiding this comment

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

I added LinkedIn SVG and fixed this error in the dev console
image

Looks good to me!

@rfay
Copy link
Member Author

rfay commented Dec 26, 2024

You're awesome, thanks!

@rfay rfay merged commit 837a959 into main Dec 26, 2024
2 checks passed
@rfay rfay deleted the 20241224_contact_and_linkedin branch December 26, 2024 21:55
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.

3 participants