Skip to content

Conversation

@lilleyse
Copy link
Contributor

A small mistake from b891542. The property inside TextureReader should be called texture not textureReader.

@lilleyse lilleyse requested a review from ptrgags June 23, 2021 20:52
@cesium-concierge
Copy link

Thanks for the pull request @lilleyse!

  • ✔️ Signed CLA found.
  • CHANGES.md was not updated.
    • If this change updates the public API in any way, please add a bullet point to CHANGES.md.
  • ❔ Unit tests were not updated.
    • Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

  • Cesium Viewer works.
  • Works in 2D/CV.
  • Works (or fails gracefully) in IE11.

@ptrgags
Copy link
Contributor

ptrgags commented Jun 23, 2021

Nice catch, @lilleyse I'll merge once CI passes!

@ptrgags ptrgags merged commit d9ab862 into master Jun 23, 2021
@ptrgags ptrgags deleted the fix-texture-reader branch June 23, 2021 21:08
@ptrgags
Copy link
Contributor

ptrgags commented Jun 23, 2021

Thanks @lilleyse!

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.

4 participants