Skip to content

fix(frontend): connect Settings DisplaySection language selector to next-intl routing (#2010) - #2034

Closed
aazminu wants to merge 2 commits into
LabsCrypt:mainfrom
aazminu:fix/2010-settings-display-language-selector
Closed

aazminu wants to merge 2 commits into
LabsCrypt:mainfrom
aazminu:fix/2010-settings-display-language-selector

Conversation

@aazminu

@aazminu aazminu commented Oct 18, 2026

Copy link
Copy Markdown

Summary

Fixes #2010
Closes #2010
/claim #2010

  • Connect DisplaySection's language selector in frontend/src/app/[locale]/settings/page.tsx to useLocale() and useRouter() / usePathname() (matching LanguageSwitcher.tsx) so selecting a language updates the active next-intl locale route.
  • Restrict the language selector options to the supported locales (en, es, tl) and remove the outdated fallback warning banner.
  • Update frontend/src/app/[locale]/settings/page.test.tsx to verify the supported locale options (en, es, tl), absence of the stale fallback banner, and router.replace locale navigation.

Settlement Details

  • USDT / USDC Wallet (EVM): 0x9a51790aC01c54184bcFa067bC38f7F0E7C6F6B6

@aazminu
aazminu force-pushed the fix/2010-settings-display-language-selector branch from 07549df to 833674f Compare October 24, 2026 06:11
@github-actions github-actions Bot added the security:malware-suspected Automated malware indicator found in a pull request label Oct 24, 2026
@github-actions

Copy link
Copy Markdown

Malware detection scan flagged this pull request

The scan found 1 indicator(s) in the PR diff. This PR is blocked until a maintainer reviews the flagged files.

  • frontend/src/app/[locale]/settings/page.tsx — hidden-appended-config-payload (high)

This is an automated triage signal; maintainers must verify whether the finding is malicious or intentional.

@aazminu
aazminu force-pushed the fix/2010-settings-display-language-selector branch from 833674f to 98d9801 Compare October 24, 2026 06:50
@K1NGD4VID K1NGD4VID closed this Oct 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

security:malware-suspected Automated malware indicator found in a pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug][Frontend] Settings DisplaySection language selector uses local dummy state and outdated language list

2 participants