Skip to content

Conversation

brnkhy
Copy link
Contributor

@brnkhy brnkhy commented Sep 10, 2018

fix an issue where TerrainFactoryBase didn't respect "none" terrain option

QA checklists

  • Add relevant code comments. Every API class and method should have <summary> description as well as description of parameters.
  • Add tests for new/changed/updated classes and methods!!!
  • Check out conventions in CONTRIBUTING.md.
  • Check out conventions in CODING-STYLE.md
  • Update the changelog
  • Update documentation.

Reviewers

@atripathi-mb @greglemonmapbox

Copy link
Contributor

@abhishektrip abhishektrip left a comment

Choose a reason for hiding this comment

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

👍

@abhishektrip abhishektrip merged commit 66f97e2 into develop Sep 11, 2018
abhishektrip added a commit that referenced this pull request Sep 13, 2018
* develop:
  Terrain section event refactor apply property changes at end of gui (#1009)
  fix an issue where TerrainFactoryBase didn't respect "none" terrain option (#1006)
  API's for runtime editing.  (#1005)
  Image section event refactor (#1002)
  Fix buildingid setting (#998)
  Map update develop merge fresh (#994)
  Improve query height method (#979)
  Mid floor calculation bug fix (#968)
  Updated changelog version to v1.4.6
  update changelog
  change ElevatedTerrainStrategy and ElevatedTerrainWithSidesStrategy classes to create flat terrain mesh on height data errors
  UWP fixes (and more) (#945)
abhishektrip added a commit that referenced this pull request Sep 14, 2018
…pdatePOC

* commit 'b88ed6d0ae1f7a55197456143f9cd25c0de18bbf':
  Map editor fix (#980)
  Terrain section event refactor apply property changes at end of gui (#1009)
  fix an issue where TerrainFactoryBase didn't respect "none" terrain option (#1006)
  API's for runtime editing.  (#1005)

# Conflicts:
#	sdkproject/Assets/Mapbox/Unity/Editor/PropertyDrawers/ElevationLayerPropertiesDrawer.cs
#	sdkproject/Assets/Mapbox/Unity/Map/AbstractMap.cs
#	sdkproject/Assets/Mapbox/Unity/SourceLayers/TerrainLayer.cs
@abhishektrip abhishektrip deleted the noneTerrainFix branch October 12, 2018 05:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants