Skip to content

Removing monobehaviour injection (no longer needed).#1

Merged
david-rhodes merged 1 commit intovector2dfrom
rhodes-use-runnable
Mar 31, 2017
Merged

Removing monobehaviour injection (no longer needed).#1
david-rhodes merged 1 commit intovector2dfrom
rhodes-use-runnable

Conversation

@david-rhodes
Copy link
Copy Markdown

Remove monobehaviour injection from factories as we have a better way to run coroutines.

Use `Runnable.Run()` for any “asynchronous” operations, rather than coroutines!
@david-rhodes david-rhodes requested a review from brnkhy March 30, 2017 21:55
@david-rhodes
Copy link
Copy Markdown
Author

@brnkhy Want to merge this into Vector2d. Take a look, please.

@brnkhy
Copy link
Copy Markdown
Contributor

brnkhy commented Mar 31, 2017

@david-rhodes looks good to me. we might need to pass down another parameter down to factories later on (to be able to remove static fields, rect&scale from map controller) but that's another issue of course.

@david-rhodes david-rhodes merged commit 1cbcc05 into vector2d Mar 31, 2017
@david-rhodes david-rhodes deleted the rhodes-use-runnable branch March 31, 2017 15:20
david-rhodes pushed a commit that referenced this pull request Mar 31, 2017
* implement Vector2d, Vector3d, Mathd and RectD classes
switch to double based structs for lat/lng/meter values

* move Vector2 classes to sdk-cs

* bumping to the proper branch for testing

* removed GeometryAsWsg84 usage and converting vector tile coordinates to unity space in VectorFeatureUnity class

* add missing VectorLayerVisualizer from previous commit

* Adding fix for Directions request (proper string formatting for Vector2d).

* fix the vector tile layer extent problem which wasn't working with traffic data

* Removing monobehaviour injection (no longer needed). (#1)

Use `Runnable.Run()` for any “asynchronous” operations, rather than coroutines!

* Pointing back to master.
wilhelmberg pushed a commit that referenced this pull request Jun 28, 2018
* add new HTTP files

* experimental MapboxAccess and define MAPBOX_EXPERIMENTAL

* check if EditorLocationProvider is set
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