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
Removes GeoIp Legacy support #15521
Removes GeoIp Legacy support #15521
Conversation
1b0a304
to
3f09bc4
Compare
0a95129
to
7d9b0f4
Compare
7d9b0f4
to
08b16de
Compare
08b16de
to
15e1c2b
Compare
|
||
it('should show geolocation admin without additional providers', async function () { | ||
testEnvironment.unloadGeoIp2 = 1; | ||
testEnvironment.save(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor suggestion: instead of a unloadGeoIp2
setting, we could have a pluginsToUnload
setting that could be re-used in other contexts for other plugins. what do you think of that idea?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
makes sense. change that.
Tested locally, seems to work for me, even going back to 3.x-dev. Left a couple comments otherwise. |
15e1c2b
to
8c1e73a
Compare
8c1e73a
to
fcafee4
Compare
Note: For users still using GeoIp Legacy, the update will switch their provider back to the default one.
fixes #13957