Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Nov 17 01:40
    yurloc opened #355
  • Oct 28 12:08
    Khodor1997 opened #354
  • Oct 27 13:36
    proyer-maxar commented #167
  • Oct 13 09:21
    dolorain closed #308
  • Oct 12 19:07
    DPatel13 reopened #353
  • Oct 12 19:07
    DPatel13 closed #353
  • Oct 12 19:07
    DPatel13 opened #353
  • Oct 12 18:59
    DPatel13 opened #352
  • Oct 12 14:17
    sgibbes commented #338
  • Oct 12 09:16
    kstratis commented #349
  • Oct 10 20:48
    iamtekson commented #310
  • Oct 10 19:21
    smeijer commented #349
  • Oct 10 18:53
    smeijer commented #349
  • Oct 10 18:14
    smeijer commented #349
  • Oct 10 18:06
    smeijer closed #350
  • Oct 10 18:06
    smeijer commented #350
  • Oct 10 17:37
    Marius-Romanus commented #349
  • Oct 10 17:34
    smeijer labeled #351
  • Oct 10 17:34
    smeijer labeled #351
  • Oct 10 17:34
    smeijer labeled #351
Daniel Smullen
@drspangle
of course i can copy it manually, but in order to maintain the installation process for the app in general, i must find some way to get the dist/ folder contents through bower, since that would be the idea behind providing a module through bower
in fact the idea behind distributing the module that way, i believe, would be to decouple the module from the build process' itself's dependencies
Stephan Meijer
@smeijer
yes, I understand of course. Just curious if that can be the reason.
Is it possible to refer external (cdn hosted for example), libraries?
Daniel Smullen
@drspangle
it is a very reasonable suggestion
to answer the cdn question, i am not sure, but maybe it is possible since i think i've done that in the line where i get material icons
but i am not sure how desirable this is
this means that my configuration file must have a static binding to that URL, so if it changes i have to update everything, whereas using bower's manifest i can automatically insert these dependencies since their path is predictable based on my bower config file
the leaflet bower package, for example, already has a dist/ folder
Stephan Meijer
@smeijer
Sorry, I have a phone call. Be right back.
Daniel Smullen
@drspangle
sure
Daniel Smullen
@drspangle

i believe your bower.json might be fixed by replacing the line

"main": "src/js/l.control.geosearch.js",

with

"main": "dist/bundle.js",

and then committing the dist/ folder inside the git repository

then $ bower install leaflet-geosearch will work out of the box
also, to answer the CDN question, it is not possible - the purpose of bower is to also manage your module's dependencies as well, so that it can be linked to them after downloading them once, rather than managing a separate version for each module
for example, if you depend on angularJS, bower will ensure that it is downloaded once, and then all other bower-installed modules which depend on it will link to it, rather than managing it's own copy
you must define these dependencies in the bower.json file
and you can control what version you depend on as well
Stephan Meijer
@smeijer
Okay; makes sense. Although I do not understand why that fixes your problem. Why can you than refer to the globals, and not when you point directly to the bundle.js. It's strange for me.
Daniel Smullen
@drspangle
it's strange for me as well, i don't have a good explanation for why i can't use bundle.js as it is
Stephan Meijer
@smeijer
I must think about this. I'm not really in favor of committing build files to a git repo. Especially since bower is already advising to start using yarn in favor of bower.
Daniel Smullen
@drspangle
indeed
it is an important decision, the issue revolves around supporting bower, which is still supported
webpack and yarn and the like are suggested for new projects
Stephan Meijer
@smeijer
true
Daniel Smullen
@drspangle
but unfortunately my project still depends on bower for building
Stephan Meijer
@smeijer
and when installing trough bower, you can specify a git branch or something?
Daniel Smullen
@drspangle
i'm not sure about that, i think you must specify a git endpoint, and you use tags to specify the version/branch
so you could use the tags to specify a particular version, but you could also use it to specify a particular branch for releases that support bower
or both of course
i.e. v2.0.0-bower
Stephan Meijer
@smeijer
what would be the command to install a particular release?
Daniel Smullen
@drspangle
you wouldn't do that from the command line - you would just do bower install leaflet-geosearch --save to automatically add the dependency, but it would download the latest version that's defined in the bower package you registered
what i would do if i wanted a specific version would be to add a dependency on that particular version in my own bower.json file, and then when i run bower install it would automatically download that one only
i believe that the tags you use on github are the way you manage the versioning directly
the semantic versioning spec is what is used, in fact the version string in the bower.json file in your package is not used
by versioning spec i mean, this is what you must use when you define your tags on the git
repo
Daniel Smullen
@drspangle
i have a doctor's appointment very shortly, so i must go for an hour or so, but i'll check in with you later about this
Daniel Smullen
@drspangle
hello
Daniel Smullen
@drspangle
@smeijer it seems that it doesn't matter which folder the bundle.js file is in, it can't be concatenated because there's something about it that prevents the objects inside being exposed
i don't know if it's related to this webpack stuff inside or what
it's not clear if that's automatically generated boilerplate
Stephan Meijer
@smeijer
Possible. Because in the node-world we all try to not expose anything to the global.
That's why "we" need these import statements :(
Which I'm very happy with by the way. But now for bower it really sucks
Daniel Smullen
@drspangle
well not just bower actually, i think the reason why i am not just using the import statements directly is because i am trying to use angular directives
Emil Malmsten
@Luxcaput_twitter
Hello! I'm trying to learn, but I have no idea on how to use this. I've downloaded and put the files in root. Can someone guide me trough the next steps?
Alma Z
@Carsak
Hello! Anybody here?
Does https://github.com/smeijer/leaflet-geosearch library support reverse geocoding?
coordinates => to address