Skip to content

Fix duplicate line and typo in Tolgee.swift#4

Merged
petrpavlik merged 2 commits intofeature/relatime-language-switchingfrom
copilot/sub-pr-3
Jan 2, 2026
Merged

Fix duplicate line and typo in Tolgee.swift#4
petrpavlik merged 2 commits intofeature/relatime-language-switchingfrom
copilot/sub-pr-3

Conversation

Copy link
Contributor

Copilot AI commented Jan 2, 2026

Code review identified two minor issues in the realtime language switching implementation.

Changes

  • Removed duplicate cdnEtags.removeAll() call in clearCaches() method
  • Fixed typo localLaunguagelocalLanguage in tuple return type and property assignments (4 occurrences)

Both issues were in /Sources/Tolgee/Tolgee.swift and did not affect functionality, but improve code quality and maintainability.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: petrpavlik <1067678+petrpavlik@users.noreply.github.com>
Copilot AI changed the title [WIP] Add option for realtime language switching feature Fix duplicate line and typo in Tolgee.swift Jan 2, 2026
Copilot AI requested a review from petrpavlik January 2, 2026 11:43
@petrpavlik petrpavlik marked this pull request as ready for review January 2, 2026 12:14
@petrpavlik petrpavlik merged commit 3694c19 into feature/relatime-language-switching Jan 2, 2026
@petrpavlik petrpavlik deleted the copilot/sub-pr-3 branch January 2, 2026 12:15
petrpavlik added a commit that referenced this pull request Jan 8, 2026
* support language switching basics

* Implement cache clearing for old app versions and add mock implementation

* Add tests for clearing old cache in MockCache and FileCache

* Enhance localization support by adding Czech and English translations, updating test cases, and refining initialization logic for locale handling

* Add language switcher support and improve localization handling in examples

* Fix language code in README examples to use Brazilian Portuguese (pt_BR)

* Support pre-fetching languages from CDN.

* add better language detection

* refactor bundle repository

* CDN language inferences now supports regions

* Enhance language resolution logic and add support for custom local language

* Add bundleForLanguageDetection parameter to Tolgee initialization

* language switching for swiftui previews is not iOS 18.4+ anymore

* cleanup

* Refine documentation for locale handling and initialization in Tolgee SDK

* Add README examples for SwiftUI and UIKit integration with Tolgee SDK

* Clarify parameter name in documentation for fetching translations

* fixes

* Add pre-fetch all languages to readme

* Fix duplicate line and typo in Tolgee.swift (#4)

* Initial plan

* Fix duplicate line and typo in variable name

Co-authored-by: petrpavlik <1067678+petrpavlik@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: petrpavlik <1067678+petrpavlik@users.noreply.github.com>

---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: petrpavlik <1067678+petrpavlik@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants