Plugin Lieux 🌍

[tobiaseigen]

@merefield

when it was smaller, users complained it wasn’t big enough.

You will never win. :rofl

If popup is the idea, I prefer the way the map pops up on the topic lists vs in the topics themselves.

The map is now finding my house properly but is getting the zip code and some parts of my address wrong.

[merefield (Robert)]

@tobiaseigen

If popup is the idea, I prefer the way the map pops up on the topic lists vs in the topics themselves.

Yup this is one of the new features.

Well maybe one day the Topic system will get a makeover …

… now back to making a living

[packman (Chris McMahon)]

Thanks for updating the plugin. I’ve only had a quick test after updating to the new version but didn’t see any problems.

[huynhthai824 (Sora &\)]

Displaying all topics on the map with Discourse Location Plugin

Hi everyone,

I’m using the Discourse Location Plugin for my real estate website. I’ve categorized listings by district/street, and each topic is a listing for sale or rent.

I want to display all these topics on the /map page. I’m using the plugin’s default map. However, when I go to a topic to edit and assign a location, the “Find Address” field is frozen.

I’m not sure what I’m missing. Any help would be greatly appreciated!

Thank you! Happy Valentine’s Day!

[merefield (Robert)]

Hey, I can provide support to businesses who sign up to one of my GOLD or PLATINUM monthly Sponsorship tiers.

There is also a one-off bug hunt option.

[Arkshine]

Robert, quick issue when running qunit tests unrelated to this plugin. It makes the tests fail.

I think writing custom_fields?.location_enabled should be enough.

[merefield (Robert)]

Sure, np,

FIX: allow non-locations tests to continue even if locations CF is not defined

[Arkshine]

Sorry Robert, actually I missed an occurrence. Should be the last:

FIX: allow non-locations tests to continue even if locations CF is not defined (again)

[Canapin (Coin-coin le Canapin)]

I used this code to add the location below the topics titles on the topic list:
javascripts/discourse/connectors/topic-list-after-title/topic-list-after-title.hbr

{{#if context.topic.location}}
  <br>
  <span class="title-location">
    {{d-icon 'map-marker-alt'}} {{location-format context.topic.location opts}}
  </span>
{{/if}}

It won’t work soon, if someone has an idea how to make it compatible with the recent version

[merefield (Robert)]

Not sure you need that anymore. It’s added below the title now.

[Canapin (Coin-coin le Canapin)]

I’m a bit puzzled. It works on Achats - Monocycle.info

But not on Trading Post - Unicyclist.com

(there are locations set on almost every topic)

Here are the modified settings on the first link

And the 2nd:

The location-related category settings are the same for both sites.

Can’t figure out why the locations don’t appear on unicyclist.com, tho I’m sure I just missed something obvious.

[Arkshine]

https://unicyclist.com/c/trading-post/18?safe_mode=no_themes

It looks like a TC interferes with the plugin here.

[Canapin (Coin-coin le Canapin)]

Yeah, I had removed the standard location icon a long time ago so I could override it with my own location component (I’m sure I had a very good reason for this ), and I forgot to revert this back after deleting my custom component.

Thanks for the help

[merefield (Robert)]

And what is more, those locations are now clickable and will pull up the map

[merefield (Robert)]

btw, on the topic of “maps” I highly recommend you (and other users of the plugin) explore the different maps available via the link in the settings:

the default one has high utility but a bit … colourless.

[merefield (Robert)]

@Canapin I just happened to browse your forum.

NB if you don’t fully Geolocate your locations, you will end up with this.

Washington is showing as one big map with no pointer.

It should be more like this:

[Canapin (Coin-coin le Canapin)]

Yeah, I know but I don’t feel like educating my users about it (I think most will only post once in the trading post category ) nor edit each location myself each time a new topic is created

I noticed this, the new posts dot is not properly aligned with the topic’s title on the topic list

[merefield (Robert)]

Looking forward to your PR

If not, I’ll look at it soon.

PS for most use cases, I’m not convinced this plugin connector is useful … it really should be after the badge. I’ll look at switching it.

[merefield (Robert)]

Fixed:

FIX: move plugin outlet for Topic List locations label

FIX: topic list location label styling

Cheers.

[dsims (Daniel)]

Trying this plugin for the first time in my local dev and noticed some issues. When I get some time I’ll see if I can work on a PR.

When editing a topic to add location, the TopicLocation is not created, so it doesn’t appear on the map. Seems due to the location custom_field string not being parsed correctly.

When viewing a category, the locations don’t appear in the topic list, because of using the “before category” outlet (there is no category outlet when viewing a category topic list). Also location_topic_status doesn’t appear to be used anywhere.