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 a beta warning warning people about the fact we're still in beta. #685

Merged
merged 3 commits into from
Sep 25, 2024

Conversation

MelissaAutumn
Copy link
Member

Fixes #684

...ignore the branch number, I'm not sure how I messed it up that bad.

Needs German strings!

Screenshots

image

@MelissaAutumn MelissaAutumn added the l10n update A string has been added or needs updating label Sep 24, 2024
@MelissaAutumn MelissaAutumn self-assigned this Sep 24, 2024
Copy link
Member

@kewisch kewisch left a comment

Choose a reason for hiding this comment

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

Here is some German for you :)

@@ -357,6 +357,21 @@
"shareMyLink": "Share my link",
"reportBug": "Report Bug"
},
"notices": {
"betaWarning": {
"heading": "As we're currently in beta, we do have some issues we're dealing with:",
Copy link
Member

Choose a reason for hiding this comment

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

Da wir aktuell in der Beta-Phase sind, hier einige Tips um Probleme zu umgehen:

(As we're currently in beta, here are a few tips to avoid problems) - I think this sounds a bit more positive than "issues"

"betaWarning": {
"heading": "As we're currently in beta, we do have some issues we're dealing with:",
"list": [
"Please check your spam folder as some emails might end up there",
Copy link
Member

Choose a reason for hiding this comment

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

Überprüfe den Spam Ordner, da einige Emails dort landen könnten

"heading": "As we're currently in beta, we do have some issues we're dealing with:",
"list": [
"Please check your spam folder as some emails might end up there",
"If Zoom meetings don't appear in your appointment invite, please disconnect and reconnect your zoom account in the {connectedAccounts} section of the settings",
Copy link
Member

Choose a reason for hiding this comment

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

Falls Zoom Meetings nicht in der Kalender-Einladung auftauchen, bitte trenne die Verbindung im {connectedAccounts} Abschnitt der Einstellungen, und stelle sie erneut her

"list": [
"Please check your spam folder as some emails might end up there",
"If Zoom meetings don't appear in your appointment invite, please disconnect and reconnect your zoom account in the {connectedAccounts} section of the settings",
"If you experience other issues, please use the {contactUs} form, or join us on our {matrixChannel}"
Copy link
Member

Choose a reason for hiding this comment

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

Sollte es noch weitere Probleme geben, nutze das {contactUs}, oder kommt in unseren {matrixChannel}

"If you experience other issues, please use the {contactUs} form, or join us on our {matrixChannel}"
],
"linkText": {
"connectedAccounts": "Connected Accounts",
Copy link
Member

Choose a reason for hiding this comment

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

Verbundene Konten

],
"linkText": {
"connectedAccounts": "Connected Accounts",
"contactUs": "contact us",
Copy link
Member

Choose a reason for hiding this comment

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

Kontaktformular

"linkText": {
"connectedAccounts": "Connected Accounts",
"contactUs": "contact us",
"matrixChannel": "Matrix Channel"
Copy link
Member

Choose a reason for hiding this comment

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

Matrix Kanal

Copy link
Member

@kewisch kewisch Sep 25, 2024

Choose a reason for hiding this comment

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

Though you might want to sway away from the technology and say something about it simply being a chat.

With "chat room" it could be

Sollte es noch weitere Probleme geben, nutze das {contactUs}, oder kommt in unseren {matrixChannel}

with matrixChannel = Chat

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks! I've stuck with matrix channel for now.

Co-authored-by: Philipp Kewisch <[email protected]>
Copy link
Collaborator

@devmount devmount left a comment

Choose a reason for hiding this comment

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

Great stuff, thanks @MelissaAutumn!

Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we add a short comment about what "user activity" is? Like what can I expect is stored here?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yep no worries. It's mostly to store the state of UI elements that are dismissed or minimized and we want that data to stick. I'll add a comment!

Copy link
Collaborator

Choose a reason for hiding this comment

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

Awesome, thanks!

Copy link
Collaborator

Choose a reason for hiding this comment

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

Nice! I forgot we already had the NoticeBar component 👏🏻

@devmount
Copy link
Collaborator

@kewisch Thank you so much for taking the time to provide German translations 👏🏻 That's usually my job here 😇

@MelissaAutumn MelissaAutumn merged commit 7614434 into main Sep 25, 2024
4 checks passed
@MelissaAutumn MelissaAutumn deleted the features/836-dismissible-beta-warning branch September 25, 2024 15:38
@kewisch
Copy link
Member

kewisch commented Sep 25, 2024

Didn't want to put you out of your job ;-) I randomly saw a notification that said German was missing, glad I could help.

@devmount
Copy link
Collaborator

No worries, great work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n update A string has been added or needs updating
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a dismissable usability warning to dashboard
3 participants