Angular-google-maps: Feature Request : Add support for drawing manager

Created on 26 Sep 2017  路  10Comments  路  Source: SebastianM/angular-google-maps

Issue description
Please refer to the below url
Google Maps Drawing Libary

Steps to reproduce and a minimal demo of the problem

_Use https://plnkr.co or similar -- try this template as a starting point: http://plnkr.co/edit/YX7W20?p=preview

_What steps should we try in your demo to see the problem?_

Current behavior

Expected/desired behavior

angular2 & angular-google-maps version

Other information

stale

Most helpful comment

Hi, I've just submitted a pull request to implement the drawing manager.

All 10 comments

Yes, please, it would be fantastic to have it.

Hi, I've just submitted a pull request to implement the drawing manager.

@Alexandre-Herve , How exciting! will this get merge into production/master soon? I have a client that wants this for next week? Can I try out you pull request some how? Many thanks

@thejustbrand I use my own work from my github fork before it is merged, but the way I do it is hacky because the published npm package doesn't look like the git repo, it is the result of a build instead.

So what I have done is create a branch on the fork where the build is versionned, pull this branch from github and replace the @agm node_module files by my own files in a postinstall script. (don't try this at home ...)

So my package.json then looks like this:

    "@agm/core": "^1.0.0-beta.2",
    "@agm/snazzy-info-window": "^1.0.0-beta.2",
    "agm": "git+https://[email protected]/demainchezmoi/angular-google-maps#drawing-manager-build",

And the postinstall script like so:

"postinstall": "rm -rf node_modules/@agm ; mkdir node_modules/@agm ; mv node_modules/agm/* node_modules/@agm/"

@Alexandre-Herve , oh i see. Probably not good for a production app then. What is the likely hood of this being merged into main AGM?
but I might give it a go in a try out locally or on a separate branch of my code base - if i can figure out how to get it running. Really excited about this!

@thejustbrand or you could build and use the built files directly in your project, bypassing npm. I have no idea, it really depends on when SebastianM will do the review, wether there will be something to fix and at what speed they release ...

@Alexandre-Herve Ok great. I'' implement it with the 'agm' in the package.json. then I have an option and other dev can use the core version without the post install script. How do you implement your post install script/cmd?

I am more than happy to do some testing for you on this branch. I can use it to implement this new proof of concept feature we have in our app and therefore testing it out in our team. Would you mind helping me get it setup or give me some pointers to get it working?
We can discuss outside of here if easier?...

Thank you @Alexandre-Herve for implementing this

Hello @Alexandre-Herve, thank you for this, I'm testing your code but I'm receiving this error: "Cannot read property 'DrawingManager' of undefined" in google-maps-api-wrapper.js line 37, the object google.maps."drawing" doesn't exist. Can you help me with that please?
隆Thanks in advanced!

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

ChrisDevinePimss picture ChrisDevinePimss  路  3Comments

mensch picture mensch  路  3Comments

supran2811 picture supran2811  路  4Comments

stot3 picture stot3  路  3Comments

Subhojit1992 picture Subhojit1992  路  3Comments