Skip to content
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.

Sign up for GitHub

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

Jump to bottom

remove(mapbox) public MapboxLayer api #8585

Merged
merged 5 commits into from Mar 6, 2024
Merged

remove(mapbox) public MapboxLayer api #8585

merged 5 commits into from Mar 6, 2024

Conversation

chrisgervang
Copy link
Collaborator

@chrisgervang chrisgervang commented Mar 6, 2024 •
edited

Closes #8584

Change List

  • Removes MapboxLayer
  • fix mapbox test apps
  • Upgrade guide
  • docs

@chrisgervang
remove(mapbox) public MapboxLayer api
3f2deb5 
Signed-off-by: Chris Gervang <chris@gervang.com>
@coveralls
Copy link

coveralls commented Mar 6, 2024 •
edited

Coverage Status

coverage: 70.67% (-0.001%) from 70.671%
when pulling 0cd65a7 on chr/deprecate-mapbox-layer
into ef324f8 on master.

@chrisgervang
chore(tests) fix mapbox test app
8890e3f 
Signed-off-by: Chris Gervang <chris@gervang.com>
@chrisgervang chrisgervang merged commit 038b823 into master Mar 6, 2024
3 checks passed
@chrisgervang chrisgervang deleted the chr/deprecate-mapbox-layer branch March 6, 2024 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Pessimistress Pessimistress Pessimistress approved these changes

@felixpalmer felixpalmer Awaiting requested review from felixpalmer

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Deprecate MapboxLayer in favor of MapboxOverlay
3 participants
@chrisgervang @coveralls @Pessimistress