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

fix: [IOBP-1100] German FAQ strings #6579

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

LeleDallas
Copy link
Contributor

Short description

This pull request corrects the display strings for the German FAQ.

List of changes proposed in this pull request

  • Update io-service-metadata version
  • Add german key to localeToLocalizedMessageKey constant
  • Added fallbackForLocalizedMessageKeys function to provide a fallback for German translations when they are not available
  • Updated locale handling to use the new fallback mechanism

How to test

Important

Align io-dev-api-server with this PR

  • Select German language, reload the app
  • Open the app
  • Tap on FAQ
  • Ensure that categories and subcategories are in german

Preview

Screen.Recording.2024-12-23.at.15.35.05.mov

@LeleDallas LeleDallas added the IO-Bonus e pagamenti IO - Bonus e pagamenti label Dec 23, 2024
@LeleDallas LeleDallas self-assigned this Dec 23, 2024
Copy link
Contributor

github-actions bot commented Dec 23, 2024

Jira Pull Request Link

This Pull Request refers to the following Jira issue IOBP-1100

Copy link

codecov bot commented Dec 23, 2024

Codecov Report

Attention: Patch coverage is 25.00000% with 9 lines in your changes missing coverage. Please review.

Project coverage is 40.57%. Comparing base (80d7bd5) to head (b7aaf58).

Files with missing lines Patch % Lines
...nce/screens/ItwIssuanceCredentialFailureScreen.tsx 0.00% 3 Missing ⚠️
...s/payments/home/components/PaymentsAlertStatus.tsx 0.00% 3 Missing ⚠️
...heckout/components/WalletPaymentFeedbackBanner.tsx 0.00% 1 Missing ⚠️
ts/hooks/useStatusAlertProps.ts 50.00% 1 Missing ⚠️
ts/utils/payment.ts 0.00% 1 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (80d7bd5) and HEAD (b7aaf58). Click for more details.

HEAD has 5 uploads less than BASE
Flag BASE (80d7bd5) HEAD (b7aaf58)
10 5
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6579      +/-   ##
==========================================
- Coverage   49.29%   40.57%   -8.73%     
==========================================
  Files        1555     1399     -156     
  Lines       32122    29455    -2667     
  Branches     7267     6625     -642     
==========================================
- Hits        15834    11950    -3884     
- Misses      16250    17475    +1225     
+ Partials       38       30       -8     
Files with missing lines Coverage Δ
ts/i18n.ts 100.00% <ø> (ø)
ts/store/reducers/__mock__/backendStatus.ts 100.00% <ø> (ø)
ts/utils/locale.ts 62.96% <100.00%> (-17.04%) ⬇️
...heckout/components/WalletPaymentFeedbackBanner.tsx 7.14% <0.00%> (-54.40%) ⬇️
ts/hooks/useStatusAlertProps.ts 60.00% <50.00%> (+2.85%) ⬆️
ts/utils/payment.ts 37.27% <0.00%> (-24.86%) ⬇️
...nce/screens/ItwIssuanceCredentialFailureScreen.tsx 9.09% <0.00%> (-0.17%) ⬇️
...s/payments/home/components/PaymentsAlertStatus.tsx 28.57% <0.00%> (-4.77%) ⬇️

... and 418 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 80d7bd5...b7aaf58. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IO-Bonus e pagamenti IO - Bonus e pagamenti
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support categories only uses italian strings
1 participant