Skip to content

chore(IT-Wallet): [SIW-4689] Remove unused i18n keys for IT-Wallet feature#8328

Open
mastro993 wants to merge 8 commits into
masterfrom
SIW-4689-remove-unused-i18n-keys
Open

chore(IT-Wallet): [SIW-4689] Remove unused i18n keys for IT-Wallet feature#8328
mastro993 wants to merge 8 commits into
masterfrom
SIW-4689-remove-unused-i18n-keys

Conversation

@mastro993

Copy link
Copy Markdown
Contributor

Warning

Depends on #8323

Short description

Removes unused keys in it locale

List of changes proposed in this pull request

  • Removed it locale unused keys

How to test

Static checks should pass

@mastro993 mastro993 self-assigned this Jul 17, 2026
@mastro993 mastro993 added IT Wallet 💀 Dead matter Graphic assets or chunks of code that are no longer referenced or needed labels Jul 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

PR Title Validation for conventional commit type

All good! PR title follows the conventional commit type.

@github-actions

github-actions Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Jira Pull Request Link

This Pull Request refers to Jira issues:

@codecov

codecov Bot commented Jul 17, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 51.51515% with 32 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.11%. Comparing base (70a2285) to head (4d9dcb6).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
.../cie/screens/ItwIdentificationCieWarningScreen.tsx 0.00% 14 Missing ⚠️
...n/cie/components/ItwCieCardReadProgressContent.tsx 0.00% 4 Missing ⚠️
...omponents/ItwPresentationCredentialStatusAlert.tsx 50.00% 2 Missing and 2 partials ⚠️
...itwallet/common/components/ItwIssuanceMetadata.tsx 0.00% 0 Missing and 2 partials ⚠️
.../ts/features/itwallet/trustmark/machine/actions.ts 0.00% 2 Missing ⚠️
.../itwallet/common/components/ItwCredentialClaim.tsx 0.00% 1 Missing ⚠️
...twallet/common/components/ItwEidLifecycleAlert.tsx 85.71% 0 Missing and 1 partial ⚠️
...s/features/itwallet/common/utils/itwClaimsUtils.ts 0.00% 1 Missing ⚠️
...ntification/spid/screens/ItwSpidIdpLoginScreen.tsx 0.00% 1 Missing ⚠️
...onents/ItwPresentationCredentialCardFlipButton.tsx 0.00% 1 Missing ⚠️
... and 1 more
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #8328      +/-   ##
==========================================
- Coverage   63.12%   63.11%   -0.02%     
==========================================
  Files        1959     1959              
  Lines       37127    37164      +37     
  Branches     6238     6254      +16     
==========================================
+ Hits        23438    23455      +17     
- Misses      12320    12337      +17     
- Partials     1369     1372       +3     
Files with missing lines Coverage Δ
...cation/common/hooks/useContinueWithBottomSheet.tsx 66.66% <100.00%> (+7.57%) ⬆️
...n/screens/ItwIdentificationModeSelectionScreen.tsx 82.35% <100.00%> (ø)
...ents/ItwPresentationNewCredentialValidityAlert.tsx 100.00% <100.00%> (ø)
...ty/components/ItwProximitySendLoadingComponent.tsx 100.00% <100.00%> (ø)
...es/itwallet/settings/screens/ItwSettingsScreen.tsx 96.00% <100.00%> (ø)
...res/wallet/components/WalletCategoryFilterTabs.tsx 83.33% <100.00%> (+0.98%) ⬆️
.../itwallet/common/components/ItwCredentialClaim.tsx 46.80% <0.00%> (-0.51%) ⬇️
...twallet/common/components/ItwEidLifecycleAlert.tsx 80.64% <85.71%> (+2.86%) ⬆️
...s/features/itwallet/common/utils/itwClaimsUtils.ts 75.71% <0.00%> (-0.55%) ⬇️
...ntification/spid/screens/ItwSpidIdpLoginScreen.tsx 8.10% <0.00%> (ø)
... and 7 more

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Harness.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@gispada gispada left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

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

Labels

💀 Dead matter Graphic assets or chunks of code that are no longer referenced or needed IT Wallet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants