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

chore(map): Map cleanup #7356

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

chore(map): Map cleanup #7356

wants to merge 10 commits into from

Conversation

VIKTORVAV99
Copy link
Member

@VIKTORVAV99 VIKTORVAV99 commented Oct 29, 2024

Issue

Closes: AVO-611

Description

Updates the bidding zone borders between the SE subzones to be more accurate, adds the channel islands and removes the NKR zone.

Double check

  • I have run pnpx prettier@2 --write . and poetry run format in the top level directory to format my changes.

@github-actions github-actions bot added translations 🗣 zone config Pull request or issue for zone configurations labels Oct 29, 2024
@VIKTORVAV99 VIKTORVAV99 requested review from a team and silkeholmebonnen and removed request for a team October 31, 2024 20:32
@VIKTORVAV99 VIKTORVAV99 requested review from tonypls and madsnedergaard and removed request for silkeholmebonnen October 31, 2024 20:33
@VIKTORVAV99 VIKTORVAV99 marked this pull request as ready for review October 31, 2024 20:33
Copy link
Member Author

Choose a reason for hiding this comment

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

The translations for this zone already exists.

Copy link
Member Author

Choose a reason for hiding this comment

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

Same for this one.

@cadeban
Copy link
Contributor

cadeban commented Nov 6, 2024

Looks good! Do we want any additional messaging for the channel islands?

Screenshot 2024-11-06 at 12 05 07

Copy link
Contributor

@cadeban cadeban left a comment

Choose a reason for hiding this comment

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

LGTM! SE borders lookin' good!

@VIKTORVAV99
Copy link
Member Author

Looks good! Do we want any additional messaging for the channel islands?

Screenshot 2024-11-06 at 12 05 07

Not just yet, the server error is due to the fact that the backend don't have access to the config files yet.

Once it does it should simply say we don't have any data for it. I'll see if I can make this more graceful in the backend at some point.

@VIKTORVAV99
Copy link
Member Author

@tonypls or @madsnedergaard do you have any objections? Otherwise I'll merge this later today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exchange config Pull request or issue for exchange configurations translations 🗣 zone config Pull request or issue for zone configurations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants