Skip to content

🐛 Fallback to proton + fix getting the Region by name#94

Merged
Zoriot merged 2 commits into
mainfrom
auto-tpll
Jun 28, 2026
Merged

🐛 Fallback to proton + fix getting the Region by name#94
Zoriot merged 2 commits into
mainfrom
auto-tpll

Conversation

@Zoriot

@Zoriot Zoriot commented Jun 28, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Zoriot added 2 commits June 28, 2026 12:38
- Replaced `getCountryFromLocationAsync` with `getRegionFromLocation` for improved modularity and fallback handling.
- Introduced `RegionLookupResult` to simplify and structure region-related responses.
- Centralized region ownership checks via `getOwnedRegionFromLocation`.
- Enhanced error handling and debugging for warp creation and editing.
- Removed redundant imports and improved method annotations for clarity and performance.

This issue was brought up by Agrro (BTE Team Czechoslovakia) after beeing teleported to france when tplling in Czechia.
…ntry

- Annotated `getByName` with `@Nullable` for better null-safety.
- Removed unused `lastRegion` variable in the `getByName` method.
- Simplified `getByName` to return `null` if no match is found.
- Minor code cleanup in `isConnected` method.
@Zoriot Zoriot requested review from a team and Jasupa June 28, 2026 11:31
@Zoriot Zoriot self-assigned this Jun 28, 2026
@Zoriot Zoriot added 🚨 high priority This currently has high priority. 🦐 small size This is a small issue. labels Jun 28, 2026
@github-actions

Copy link
Copy Markdown

Qodana for JVM

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

@Jasupa Jasupa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@Zoriot Zoriot merged commit 8dab8f5 into main Jun 28, 2026
4 checks passed
@Zoriot Zoriot deleted the auto-tpll branch June 28, 2026 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚨 high priority This currently has high priority. 🦐 small size This is a small issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants