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
Add support for OS Maps API #3328
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3328 +/- ##
==========================================
+ Coverage 82.21% 82.27% +0.05%
==========================================
Files 320 324 +4
Lines 21375 21356 -19
Branches 3214 3213 -1
==========================================
- Hits 17574 17571 -3
+ Misses 2753 2742 -11
+ Partials 1048 1043 -5
Continue to review full report at Codecov.
|
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.
One question, but looks great overall! 👍
OS Maps key, layer, URL, and whether it's premium or not can be provided on a per-cobrand basis. In order to do that I had to switch the OSM map classes to use Moo (so e.g. number of zoom levels could look at configuration to work itself out), which hopefully tidies it up a bit at the same time.
The other base change was to store the cobrand being used to set a map class, not just the map class. This is so that, if you are in the scenario where someone is generating a map on a OS-Maps-API-using cobrand, for a report that was generated on a non-OS-Maps-API-using cobrand, it would still be able to access the right information for the API key etc.