Skip to content

Upgrade frontend dependencies#144

Merged
luqmaan merged 4 commits intomasterfrom
143-deps
Mar 1, 2018
Merged

Upgrade frontend dependencies#144
luqmaan merged 4 commits intomasterfrom
143-deps

Conversation

@luqmaan
Copy link
Copy Markdown
Contributor

@luqmaan luqmaan commented Mar 1, 2018

Major changes:

#143

Things seem to work at a high level.

One bug I've noticed is that crossing items don't show up in the sidebar until you pan the map. I'm not sure how to fix this.

Major changes:

- react 16 => 16.2
- mapbox-gl-style-spec 9 => 11
- react-apollo 1 => 2
  - required some code changes to the middleware and import path to gql
  - https://www.apollographql.com/docs/react/2.0-migration.html
- react-mapbox-gl 2 => 3
  - required some changes to the layerOptions prop
  - https://github.com/alex3165/react-mapbox-gl/releases/tag/v3.1.0

 #143
Running into this issue: graphql/graphql-js#1248.

Need to wait for this PR to get merged: facebook/create-react-app#4085.
@luqmaan luqmaan merged commit 4694655 into master Mar 1, 2018
@luqmaan luqmaan deleted the 143-deps branch March 1, 2018 20:06
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.

1 participant