Skip to content

MPP-2083: Retire eu_country_expansion flag, frontend#3778

Merged
jwhitlock merged 1 commit intomainfrom
adopt-eu-expansion-frontend-mpp-2083
Aug 15, 2023
Merged

MPP-2083: Retire eu_country_expansion flag, frontend#3778
jwhitlock merged 1 commit intomainfrom
adopt-eu-expansion-frontend-mpp-2083

Conversation

@Vinnl
Copy link
Collaborator

@Vinnl Vinnl commented Aug 15, 2023

Followup to #3745, to remove the flag from the UI code as well. Assumes the flag to always be active.

Followup to 5ec8351. Assumes the
flag to always be active.
@Vinnl Vinnl added the Review: µ Code review time: 5 minutes or less label Aug 15, 2023
@Vinnl Vinnl requested review from jwhitlock and lloan August 15, 2023 13:51
@Vinnl Vinnl self-assigned this Aug 15, 2023
Copy link

@bcolsson bcolsson left a comment

Choose a reason for hiding this comment

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

Approving without comment as this doesn't affect l10n.

Copy link
Contributor

@jwhitlock jwhitlock left a comment

Choose a reason for hiding this comment

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

Thanks @Vinnl! I'm re-running the front-end tests, looks like another one that needs more than 5 seconds.

@jwhitlock jwhitlock added this pull request to the merge queue Aug 15, 2023
@jwhitlock jwhitlock changed the title Retire eu_country_expansion flag, frontend MPP-2083: Retire eu_country_expansion flag, frontend Aug 15, 2023
Merged via the queue into main with commit 1750d60 Aug 15, 2023
@jwhitlock jwhitlock deleted the adopt-eu-expansion-frontend-mpp-2083 branch August 15, 2023 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Review: µ Code review time: 5 minutes or less

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants