Skip to content

feat: add contact form #257

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

Merged
merged 1 commit into from
Feb 24, 2024
Merged

feat: add contact form #257

merged 1 commit into from
Feb 24, 2024

Conversation

anbraten
Copy link
Member

No description provided.

Copy link

github-actions bot commented Feb 23, 2024

♻️ PR Preview 486aaab has been successfully destroyed since this PR has been closed.

🤖 By surge-preview

Copy link
Member

@lukashass lukashass left a comment

Choose a reason for hiding this comment

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

I guess this is only a good UX when the mailto handler is configured, otherwise you write something to then notice it doesn't work.

Since we can ignore desktop it might be ok though.

@anbraten
Copy link
Member Author

anbraten commented Feb 24, 2024

Sure in the end you need to have a browser supporting the mailto link type (seems to be the case for all major ones however). Desktop works for me as well?!

@anbraten anbraten merged commit 167ab26 into main Feb 24, 2024
@anbraten anbraten deleted the contact branch February 24, 2024 01:39
@@ -52,5 +51,8 @@
"light": "Hell",
"dark": "Dunkel",
"next_stop": "Nächster Stop:",
"ok": "Ok"
"ok": "Ok",
"contact_title": "Möchtest du Feedback geben, eine Idee vorschlagen, ein Problem melden oder einfach nur Danke sagen? Wir freuen uns über deine E-Mail!",

Choose a reason for hiding this comment

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

Minor thing, but I think this should read "Nachricht" instead of E-Mail.

"ok": "Ok"
"ok": "Ok",
"contact_title": "Möchtest du Feedback geben, eine Idee vorschlagen, ein Problem melden oder einfach nur Danke sagen? Wir freuen uns über deine E-Mail!",
"send_email": "E-Mail senden",

Choose a reason for hiding this comment

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

here as well

@@ -52,5 +51,9 @@
"light": "Light",
"dark": "Dark",
"next_stop": "Next stop:",
"ok": "Ok"
"ok": "Ok",
"contact_title": "Would you like to share feedback, propose an idea, report an issue, or simply express gratitude? Your email would be greatly appreciated!",

Choose a reason for hiding this comment

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

message instead of email

"ok": "Ok",
"contact_title": "Would you like to share feedback, propose an idea, report an issue, or simply express gratitude? Your email would be greatly appreciated!",
"contact_email_body": "Hi Kiel-Live Team 👋,\n\n",
"send_email": "Send email",

Choose a reason for hiding this comment

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

same

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

Successfully merging this pull request may close these issues.

3 participants