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

[core] Change definition of "complete" so offline packs created before #11055 can complete. #14188

Merged
merged 3 commits into from
Mar 21, 2019

Conversation

julianrex
Copy link
Contributor

@julianrex julianrex commented Mar 21, 2019

Fixes a bug that causes offline packs created prior to #11055 to be marked as MGLOfflinePackStateInactive on iOS/macOS.

/cc @tobrun

@julianrex julianrex requested a review from 1ec5 as a code owner March 21, 2019 16:09
@julianrex julianrex requested a review from a team March 21, 2019 16:09
@julianrex julianrex added the Core The cross-platform C++ core, aka mbgl label Mar 21, 2019
Copy link
Contributor

@tmpsantos tmpsantos left a comment

Choose a reason for hiding this comment

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

I'm writing a unit test for this. But I can verify the migration issue on command line already.

@julianrex
Copy link
Contributor Author

I'm writing a unit test for this. But I can verify the migration issue on command line already.

Fantastic - thank you! I'll go ahead and merge this one so it can get into the beta, but would love the unit test in for the stable release.

@zugaldia
Copy link
Member

@tobrun is a similar change required on Android?

@julianrex
Copy link
Contributor Author

@zugaldia @tobrun this change was in core - the bug would also affect Android.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Core The cross-platform C++ core, aka mbgl
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants