Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

[core] Prepare changelog for maps-v1.3.0 (2020.02-release-vanillashake) release #16247

Conversation

alexshalamov
Copy link
Contributor

Release maps-v1.3.0

Changes since maps-v1.2.1 release

🐞 Bug fixes

  • [core] Fix offline region download freezing (#16230)

    Downloaded resources are put in the buffer and inserted in the database in batches.

    Before this change, the buffer was flushed only at the network response callback and thus it never got flushed if the last required resources were present locally and did not initiate network requests - it caused freezing.

    Now the buffer is flushed every time the remaining resources container gets empty.

✨ New features

  • [core] Add Layer::serialize() method (#16231)

    New method allows serialization of a layer into a Value type, including all modifications done via runtime style API. New method is also an enabler for Style object serialization (sources, layers, etc).

  • [android] Add jni binding for min and max pitch (#16236)

  • [offline] Offline tool does not hang on 404 error (#16240)

    The missing resource gets skipped and teh offline region download continues.

⚠️ Breaking changes
  • Changes to mbgl::FileSourceManager::getFileSource() (#16238)

    It returns now mbgl::PassRefPtr<FileSource> (previously was std::shared_ptr<FileSource>) in order to enforce keeping the strong reference to the returned object.

    Breaking code example:
    auto fs = FileSourceManager::getFileSource(); fs->..

    Posible fix:
    std::shared_ptr<FileSource> fs = ;

  • The mbgl::OnlineFileSource class cannot be used directly (#16238)

    Clients must use the parent mbgl::FileSource interface instead.

    Breaking code example:
    std::shared_ptr<OnlineFileSource> onlineSource = std::static_pointer_cast<OnlineFileSource>(FileSourceManager::get()->getFileSource(..));

    Possible fix:
    std::shared_ptr<FileSource> onlineSource = FileSourceManager::get()->getFileSource(..);

Launch Checklist

  • briefly describe the changes in this PR
  • tagged @mapbox/maps-android @mapbox/maps-ios @mapbox/core-sdk if this PR adds or updates a public API

Juha Alanen and others added 30 commits February 28, 2020 16:13
Also remove some unnecessary files.
Use the latest Xcode 11.3.1 and run the tests on
iPhone 11 with the latest iOS 13.3.
…#16193)

* Revert "[ios, macos] Include SKU token for offline requests (#16130) (#16133)"

This reverts commit d85ce77.

* [ios] Keep skipping docs for ios-release
GCC complains about this variable being potentially not initialized.
* [android] and jni interface for source zoom delta

* handle nullopt in setter and getter

* Use jni::Integer
and get rid of `SpriteLoaderWorker`, use `scheduleAndReplyValue()` API instead.
This will move geojson-vt-cpp and shelf-pack-cpp
to mapbox-base collection of libraries.
Downloaded resources are put in the buffer and inserted in the database in batches.

Before this change, the buffer was flushed only at the network response callback
and thus it never got flushed if the last required resources were present locally
and did not initiate network requests -> it caused freezing.

Now the buffer is flushed every time the remaining resources container gets empty.
Thus we do not need to include specific headers to use `{set|get}Property`
API with key constants.
pozdnyakov and others added 24 commits February 28, 2020 16:16
Add a script that for everything we link against
mbgl-core, it will check the license and generate
a file all the dependencies listed with the respective
authors, project urls and license.
They have a different license from the rest of the code
and this will make it explicit. Also the script for generating
the license will dectect them correctly.
Also make sure it will be always up to date
by using the nitpick bot to enforce it.
For historical reasons, developers may come here when looking for the Mapbox Maps SDK. Direct them to the right repository so they don’t get confused thinking Mapbox only provides a C++ library.
* [android] Add jni binding for min and max pitch

* Update CHANGELOG.md
@alexshalamov alexshalamov requested a review from zmiao February 28, 2020 14:40
@alexshalamov alexshalamov marked this pull request as ready for review February 28, 2020 14:40
@alexshalamov alexshalamov requested a review from 1ec5 as a code owner February 28, 2020 14:40
@alexshalamov
Copy link
Contributor Author

Will be better to create new branch for out-of-release cycle release.

@alexshalamov alexshalamov deleted the alexshalamov_release_maps-v1.3.0 branch February 28, 2020 14:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants