Locations Plugin ๐ŸŒ

do we have any chance to have this feature request out of the waitlist after one year? Thanks!

Hi @C_X as already stated, this is a (currently free) Pavilion plugin.

All new features need to be sponsored.

If you have budget you could hire me to build it for you.

Alternatively, you could PR and I will review.

Thanks for your interest.

oh thanks @merefield ! sry I was not able to go through the whole thread to comprehend how things work here.

But yeah understood this is some feature up to sponsorship. Iโ€™ll reach out when I get to there.

Thanks in advance!

2๊ฐœ์˜ ์ข‹์•„์š”

Hi @merefield, Iโ€™m seeing the following admin notice since upgrading to the latest Discourse version:

[Admin Notice] One of your themes or plugins needs updating for compatibility with upcoming Discourse core changes. (id:discourse.fontawesome-6-upgrade)

@Arkshine mentioned here

that this might be related to the Locations Plugin and that map-marker-alt needs to be updated to location-dot.

So Iโ€™m bringing this up here. :wink:

2๊ฐœ์˜ ์ข‹์•„์š”

Feel free to submit a PR.

I will make a PR, Robert. :+1:


There are others deprecations to fix, and also one error I encountered, but for font awesome deprecation, here we go:

https://github.com/paviliondev/discourse-locations/pull/119

3๊ฐœ์˜ ์ข‹์•„์š”

Hello

Iโ€™m having a problem loading the map when I switch from one map to another.

Explanations:
Each map is in a different category

If I change category by category page or by complete URL, everything is OK

The problem occurs if I go through the sidebar.
The sidebar uses a partial URL:

Do you have a solution?

Thanks for your great plugin!


Problem encountered here:

categories with map:

1๊ฐœ์˜ ์ข‹์•„์š”

how did u got this working with launcher i am getting error as this thread
bundle exec rake db:migrateโ€™ failed with return

Looks like an extensive fix for this awesome plugin is in the pipeline:

https://github.com/paviliondev/discourse-locations/pull/120

Thanks again @merefield for the time and energy you put into maintaining this highly valuable addition to the Discourse ecosystem.

10๊ฐœ์˜ ์ข‹์•„์š”

Thank you! I am a bit out of business when it comes to programming. :wink:

With a newer version of Discourse I installed/updated yesterday, the message changed:

[Notice for Administrators] One of your themes or plugins needs to be updated to ensure compatibility with upcoming Discourse core changes. (id:discourse.hbr-topic-list-overrides) Identified plugin: โ€œdiscourse-locationsโ€

Maybe it still means the same or maybe other things turned up.

2๊ฐœ์˜ ์ข‹์•„์š”

Any idea when this might land, Rob? Just so I know if updating my Locations dependent site is likely to be feasible in the next week or so.

2๊ฐœ์˜ ์ข‹์•„์š”

Hi Nathan.

The existing (quite large) PR broke because of the new draft system which dropped, so itโ€™s been delayed and Iโ€™ll have to try to resolve that.

I will be releasing the PR once ready with an opportunity for the community and organisations to sponsor my work to help guarantee that this plugin gets maintained in a timely fashion.

3๊ฐœ์˜ ์ข‹์•„์š”

Today Iโ€™m releasing an update to bring compatibility with the fantastic new Glimmer Topic List system, modernisation and a host of other improvements and compatibility changes.

You must be on latest and enable Glimmer Topic Lists.

https://github.com/merefield/discourse-locations/pull/120

If you think Locations is awesome and I did a great job, you can buy me a coffee :coffee:, and if Locations is important to your business or organisation, please consider sponsoring me on one of the higher tiers:medal: . This will all help to guarantee timely updates to Locations and its future.

Release 6.8.0

  • FEATURE: you can now click on a Topic location on the Topic List and it will bring up a map in a modal showing the location. :map: :rocket:
  • FEATURE: new setting location_short_names removes the address from the location display for Topics, allow you to tailor what is shown.
  • COMPATIBILITY: resolves a host of deprecations to maintain compatibility with core:
    • Topic List front end API changes
    • Fontawesome icon name changes
    • New Topic Draft system.
  • FIX: Topic Locations: when setting a Topic Location via selector (not Input Fields), this resulted in a blank Location being displayed.
  • FIX: formatting of Location Selector is much much nicer.
10๊ฐœ์˜ ์ข‹์•„์š”

Cool! Congrats on getting this update in and thanks for writing release notes. I think this is a great idea and Iโ€™d be glad to see more plugin developers take the time to do this.

Now that Iโ€™ve bought you coffee already, itโ€™s time for me to take your plugin for a spin. :wink:

4๊ฐœ์˜ ์ข‹์•„์š”

Thanks Tobias! Appreciated! All feedback welcome!

3๊ฐœ์˜ ์ข‹์•„์š”

It might be helpful to add โ€œquick startโ€ section to docs that covers the basics of initial setup. Two things that took me a minute to understand:

  • The only way to allow people to start adding locations to topics is by editing the category settings for each category. It wonโ€™t just automatically allow it for all categories.
  • The defaults work out of the box for testing/low volume usage, using nominatim as the geocoding provider.

A โ€œtroubleshooting tipsโ€ section might also be helpful.

I am able to get locations to work to put users on a map. Cool! :slight_smile:

I am not able to get locations to work in topics. Not so cool! :upside_down_face:

I am able to select the popup to add a location, type it in, select Find Address, select the found address. But then Selecting Done does nothing. There is a bunch of stuff showing up on the javascript console:

3๊ฐœ์˜ ์ข‹์•„์š”

This is why this plugin needs proper funding.

I had closed out a multitude of deprecations and the console was clear, but looks like there are now further changes required to be compatible with core. :sweat_smile:

I suspect the composer changes related to drafts might be at the root of this, Iโ€™ll take a look at this when I have time.

4๊ฐœ์˜ ์ข‹์•„์š”

Turned out it was my fault :blush:

https://github.com/merefield/discourse-locations/commit/88fc9bde88f5fdfe3bd3c4c59c2563d0744a0fa5

https://github.com/merefield/discourse-locations/commit/3f5d27f83c5f2a9535a54733aea972016df0ec52

Please update and try again.

5๊ฐœ์˜ ์ข‹์•„์š”

That worked! Nice job. Iโ€™ll play with this for a bit and see how far I can take it on my sites. I have one site I run for my neighborhood, and it might be interesting to show everybody associated with their home.

On my site, the map is a bit wide for the topic space, covering the admin wrench.

2๊ฐœ์˜ ์ข‹์•„์š”

itโ€™s kind of a modal, itโ€™s not supposed to stay up - you are just supposed to be peeking, then collapsing it to carry on with your day. It definitely hasnโ€™t been a requirement to combine the map functionality with admin tools :slight_smile:

when it was smaller, users complained it wasnโ€™t big enough.

3๊ฐœ์˜ ์ข‹์•„์š”