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.

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

Use localized log message #14475

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Use localized log message #14475

wants to merge 1 commit into from

Conversation

surinder-tsys
Copy link
Collaborator

@surinder-tsys surinder-tsys commented Jan 30, 2025 •
edited
Loading

Use localized log message from library for Snackbar, Toast or any other UI places.

  • Tests written, or not not needed

@github-actions GitHub Actions
Copy link

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/14475.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

@AndyScherzinger AndyScherzinger added this to the Nextcloud App 3.31.0 milestone Jan 30, 2025
@github-actions GitHub Actions
Copy link

Codacy

Lint

TypemasterPR
Warnings5555
Errors33

SpotBugs

CategoryBaseNew
Bad practice6565
Correctness5757
Dodgy code294294
Experimental11
Internationalization77
Malicious code vulnerability11
Multithreaded correctness77
Performance5351
Security1818
Total503501

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tobiasKaminsky tobiasKaminsky Awaiting requested review from tobiasKaminsky

@alperozturk96 alperozturk96 Awaiting requested review from alperozturk96

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
3. to review
Projects
None yet
Milestone
Nextcloud App 3.31.0
Development

Successfully merging this pull request may close these issues.
2 participants
@surinder-tsys @AndyScherzinger