Skip to content

Fix notification issue #121

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.

Sign up for GitHub

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

Jump to bottom
Merged
Flajt merged 8 commits into from
Mar 17, 2024
Merged

Fix notification issue #121

Flajt merged 8 commits into from
Mar 17, 2024

Conversation

@Flajt
Copy link
Owner

@Flajt Flajt commented Mar 17, 2024

  • Should resolve issues with missing translations in the foreground notifications from blocs using tr("my.key") instead of "my.key".tr()
  • Might fix missing Sentry DSN issue by calling dotenv.load before initSentry
  • Resolves issue regarding translations in forground notifications directly #210

@Flajt Flajt merged commit 4a48d6b into dev Mar 17, 2024
@Flajt Flajt mentioned this pull request Mar 17, 2024
Merged
@Flajt Flajt deleted the fix-notification-issue branch March 20, 2024 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Flajt