Skip to content

fix: maps examples#1928

Merged
bietkul merged 43 commits intofeat/upgrade-maps-libfrom
fix/maps-examples
Apr 20, 2022
Merged

fix: maps examples#1928
bietkul merged 43 commits intofeat/upgrade-maps-libfrom
fix/maps-examples

Conversation

@mohdashraf010897
Copy link
Copy Markdown
Member

@mohdashraf010897 mohdashraf010897 commented Apr 12, 2022

PR Type Fix

Description The PR fixes the maps examples post the lib update to @react-google-maps/api.

📔 Notion

https://www.loom.com/share/55130ce498684b41a0f309187a06faad

Note: Please merge #1925 to next branch and then merge next to this branch prior to reviewing.

@bietkul
Copy link
Copy Markdown
Contributor

bietkul commented Apr 18, 2022

@mohdashraf010897 Share a demo for all the examples you've updated. Explain the code changes if needed.

@mohdashraf010897
Copy link
Copy Markdown
Member Author

@mohdashraf010897 Share a demo for all the examples you've updated. Explain the code changes if needed.

https://www.loom.com/share/55130ce498684b41a0f309187a06faad

@bietkul bietkul changed the base branch from next to feat/upgrade-maps-lib April 20, 2022 06:57
@bietkul bietkul merged commit 21b7e6c into feat/upgrade-maps-lib Apr 20, 2022
bietkul added a commit that referenced this pull request Apr 27, 2022
* fix: resolve conflicts

* migrate to @react-google-maps/api

* remove unwanted dependencies

* feat: added MarkerWithLabel component

* refactor: remove unused maps lib

* refactor: remove unused props

* fix: assign correct proptypes

* fix: assign correct proptypes

* feat: use  mapKey from ReactiveBase to load maps script

* fix(maps): load script onlywhen required for ReactiveGoogleMap

* feat: added ScriptLoader HOC for google script loading

* feat: use ScriptLoader HOC for google dependent components

* fix(maps): GeoDistanceDropdown renderItem null handling

* reset unwanted change

* fix(maps): synchronise script loading mechanism for multiple components

* feat: add redux support for script loading

* feat(map): add libraries prop for reactivegooglemap to load different libraries

* fix(maps): issue with exposing map instance to user

* fix: script loading logic

* fix(maps): labelAnchor offset reset to 0-0

* fix(maps): add null safety check for window object

* fix(maps): use action creators instead of inline actions

* fix(web): sync reactivecore package version

* feat(Web): define mapLibraries prop in ReactiveBase

* fix(maps): pass libraries prop as required

* feat(maps): consume mapLibraries prop from config

* fix(maps): remove redundant prop passing

* sync submodules

* fix: remove package-lock

* feat(maps): render api changes for map components (#1925)

* refactor(weB): import util methods from reactive lib

* sync reactivecore post refacotring util methods

* feat: accept render prop instead of renderAllData for ReactiveGoogleMap and ReactiveOpenStreet

* fix: rename renderData prop to renderItem

* reafctor(web): import util from reactivecore and update lib version

* feat(maps): add onData callback event to GeoDistance-Dropdown/Slider

* feat: add render and renderITem props

* fix: render and renderItem prop usage for GeoDistanceDropdown

* fix: GeoDistanceDropdown renderItem method to handle null condition

* fix(maps): GeoDistanceDropdown renderItem null handling

* fix(maps): api for GeoDistanceDropdown renderItem prop

* v3.29.2-preview.1

* v3.1.0-preview.1

* v3.1.0-preview.2

* v3.1.0-preview.3

* v3.29.2-preview.2

* v3.29.2-preview.3

* fix(web): bad imports from wrong path

* fix: versioining

* fix(maps): revert package.json changes

* fix(maps): ReactiveMap example package.json conflict resolution

* fix: maps examples (#1928)

* refactor(weB): import util methods from reactive lib

* sync reactivecore post refacotring util methods

* feat: accept render prop instead of renderAllData for ReactiveGoogleMap and ReactiveOpenStreet

* fix: rename renderData prop to renderItem

* reafctor(web): import util from reactivecore and update lib version

* feat(maps): add onData callback event to GeoDistance-Dropdown/Slider

* feat: add render and renderITem props

* fix: render and renderItem prop usage for GeoDistanceDropdown

* fix: GeoDistanceDropdown renderItem method to handle null condition

* fix(maps): GeoDistanceDropdown renderItem null handling

* fix(maps): GeoDistanceDropdown example

* fix(maps): GeoDistanceSlider example

* fix: airbeds site demo example

* fix(maps): api for GeoDistanceDropdown renderItem prop

* removed external google script calls from examples

* fix: weather site demo

* fix: polyline example

* v3.29.2-preview.1

* v3.1.0-preview.1

* v3.1.0-preview.2

* v3.1.0-preview.3

* v3.29.2-preview.2

* v3.29.2-preview.3

* fix(web): bad imports from wrong path

* sync with render-api changes

* fix: versioining

* fix(maps): revert package.json changes

* fix(maps): ReactiveMap example package.json conflict resolution

* fix(maps-examples): fix map examples with new api

* add emotion dependency to meetup demo

* chore: update sub packages

* chore: update lock file

* chore: upgrade reactivecore

* fix(web): failing test case

Co-authored-by: bietkul <kuldepsaxena155@gmail.com>
siddharthlatest pushed a commit that referenced this pull request Apr 29, 2022
* fix: resolve conflicts

* migrate to @react-google-maps/api

* remove unwanted dependencies

* feat: added MarkerWithLabel component

* refactor: remove unused maps lib

* refactor: remove unused props

* fix: assign correct proptypes

* fix: assign correct proptypes

* feat: use  mapKey from ReactiveBase to load maps script

* fix(maps): load script onlywhen required for ReactiveGoogleMap

* feat: added ScriptLoader HOC for google script loading

* feat: use ScriptLoader HOC for google dependent components

* fix(maps): GeoDistanceDropdown renderItem null handling

* reset unwanted change

* fix(maps): synchronise script loading mechanism for multiple components

* feat: add redux support for script loading

* feat(map): add libraries prop for reactivegooglemap to load different libraries

* fix(maps): issue with exposing map instance to user

* fix: script loading logic

* fix(maps): labelAnchor offset reset to 0-0

* fix(maps): add null safety check for window object

* fix(maps): use action creators instead of inline actions

* fix(web): sync reactivecore package version

* feat(Web): define mapLibraries prop in ReactiveBase

* fix(maps): pass libraries prop as required

* feat(maps): consume mapLibraries prop from config

* fix(maps): remove redundant prop passing

* sync submodules

* fix: remove package-lock

* feat(maps): render api changes for map components (#1925)

* refactor(weB): import util methods from reactive lib

* sync reactivecore post refacotring util methods

* feat: accept render prop instead of renderAllData for ReactiveGoogleMap and ReactiveOpenStreet

* fix: rename renderData prop to renderItem

* reafctor(web): import util from reactivecore and update lib version

* feat(maps): add onData callback event to GeoDistance-Dropdown/Slider

* feat: add render and renderITem props

* fix: render and renderItem prop usage for GeoDistanceDropdown

* fix: GeoDistanceDropdown renderItem method to handle null condition

* fix(maps): GeoDistanceDropdown renderItem null handling

* fix(maps): api for GeoDistanceDropdown renderItem prop

* v3.29.2-preview.1

* v3.1.0-preview.1

* v3.1.0-preview.2

* v3.1.0-preview.3

* v3.29.2-preview.2

* v3.29.2-preview.3

* fix(web): bad imports from wrong path

* fix: versioining

* fix(maps): revert package.json changes

* fix(maps): ReactiveMap example package.json conflict resolution

* fix: maps examples (#1928)

* refactor(weB): import util methods from reactive lib

* sync reactivecore post refacotring util methods

* feat: accept render prop instead of renderAllData for ReactiveGoogleMap and ReactiveOpenStreet

* fix: rename renderData prop to renderItem

* reafctor(web): import util from reactivecore and update lib version

* feat(maps): add onData callback event to GeoDistance-Dropdown/Slider

* feat: add render and renderITem props

* fix: render and renderItem prop usage for GeoDistanceDropdown

* fix: GeoDistanceDropdown renderItem method to handle null condition

* fix(maps): GeoDistanceDropdown renderItem null handling

* fix(maps): GeoDistanceDropdown example

* fix(maps): GeoDistanceSlider example

* fix: airbeds site demo example

* fix(maps): api for GeoDistanceDropdown renderItem prop

* removed external google script calls from examples

* fix: weather site demo

* fix: polyline example

* v3.29.2-preview.1

* v3.1.0-preview.1

* v3.1.0-preview.2

* v3.1.0-preview.3

* v3.29.2-preview.2

* v3.29.2-preview.3

* fix(web): bad imports from wrong path

* sync with render-api changes

* fix: versioining

* fix(maps): revert package.json changes

* fix(maps): ReactiveMap example package.json conflict resolution

* fix(maps-examples): fix map examples with new api

* add emotion dependency to meetup demo

* chore: update sub packages

* chore: update lock file

* chore: upgrade reactivecore

* fix(web): failing test case

Co-authored-by: bietkul <kuldepsaxena155@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants