merefield
February 26, 2026, 2:29pm
1224
Maintenance release:
main ← maintenance_release
merged 11:09PM - 25 Feb 26 UTC
## New Features
- Added a new enum setting: location_ip_auto_lookup_mode wi… th options:
- disabled
- posting
- login_and_posting
- Wired automatic IP-based location lookup into login flow (on(:user_logged_in)) when mode is login_and_posting.
- Added helper methods in plugin.rb to centralize IP auto-lookup mode/eligibility checks and enqueue behavior.
## Fixes
- Fixed composer default location initialization so user geo location is correctly prefilled when location_topic_default = user.
- Fixed category default-map rendering regression:
- category routes now render the map template safely when filter resolves to map
- avoids blank page / broken SPA behavior when category default view is map
- Fixed category map data loading by passing category context into the map template/component.
- Fixed map fallback fetch path to absolute /map.json to avoid route-relative request issues.
- Standardized users map route to users-map and removed legacy users_map usage paths.
## Configuration
- Introduced location_ip_auto_lookup_mode in config/settings.yml.
- Bumped plugin version to 7.1.6.
- Synced plugin JS devDependencies/config with core lint/tooling expectations.
## Code Formatting
- Migrated/modernized frontend code to satisfy current core lint rules:
- import path updates (discourse-i18n, moved package paths, etc.)
- removed deprecated patterns and jQuery usage
- template strict-mode migrations where required
- formatter/lint compliance cleanup across JS/GJS/HBS
- Updated prettier handling for vendored Leaflet assets to keep CI formatting checks clean.
## New Tests
- Added system spec: spec/system/composer_default_location_spec.rb
Validates composer initializes location from user location under the "user" default setting.
- Added system spec: spec/system/category_default_map_view_spec.rb
Validates category default map view renders map container without route error.
- Added request spec: spec/requests/users_map_controller_spec.rb
Covers canonical /locations/users-map route behavior.
- Extended request coverage for IP lookup posting/login mode behavior.
- Updated QUnit acceptance tests to assert intended behavior with stricter assertions and stable selectors.
followed by fixes:
main ← copilot/fix-35533304-102999061-d25577a9-2d35-4b64-9e45-583c49295ad7
merged 02:05PM - 26 Feb 26 UTC
- [x] Analyze root cause: missing `.replace-location` CSS wrapper in both user c… ard connector and profile component templates
- [x] Fix `components/replace-location.gjs`: add `.replace-location` wrapper and update `showUserLocation` to use `model.geo_location`
- [x] Fix `connectors/user-card-location-and-website/replace-location.gjs`: add `.replace-location` wrapper
- [x] Add system tests for profile geo location visibility, native location hiding, website display, and user card location display
- [x] Bump patch version to 7.1.7
- [x] Fix CI: use `fab!(:user_with_location, :user)` shorthand (Rubocop Discourse/FabricatorShorthand)
- [x] Fix failing system test: replace fragile CSS-visibility check with content-based check
- [x] Fix test setup error: use `user_profile.update!(location: ...)` instead of `user.update!(location: ...)` — location is on UserProfile not User
- [x] Align user card and profile system tests: add "hides native location" and "shows website" scenarios to user card describe block, matching the full scope of the profile tests (selector differences only reflect the different component markup)
<details>
<summary>Original prompt</summary>
>
> ----
>
> *This section details on the original issue you should resolve*
>
> <issue_title>Locations are not showing on profile and user card</issue_title>
> <issue_description>Since merefield/discourse-locations#155 locations are not showing on user card and profile page like they did before.
>
> The code should be hiding core location profile information but showing website.
>
> In the old code this was handled well, but seems have been lost during maintenance work.
>
> Check code prior to merefield/discourse-locations#155 to confirm how it should work.
>
> We should add system tests to ensure visibility of geo location and that we are hiding standard profile location field whilst exposing website from profile if any. </issue_description>
>
> ## Comments on the Issue (you are @copilot in this section)
>
> <comments>
> </comments>
>
</details>
- Fixes merefield/discourse-locations#156
---
✨ Let Copilot coding agent [set things up for you](https://github.com/merefield/discourse-locations/issues/new?title=✨+Set+up+Copilot+instructions&body=Configure%20instructions%20for%20this%20repository%20as%20documented%20in%20%5BBest%20practices%20for%20Copilot%20coding%20agent%20in%20your%20repository%5D%28https://gh.io/copilot-coding-agent-tips%29%2E%0A%0A%3COnboard%20this%20repo%3E&assignees=copilot) — coding agent works faster and does higher quality work when set up for your repo.
main ← fix_review_followup
merged 03:09PM - 26 Feb 26 UTC
- Strengthened the user-card system spec to assert rendered location label text … (city), not just element presence.
- Added the required blank line after the magic comment in the location visibility spec for RuboCop compliance.
- Strengthened the profile-page system spec to assert formatted location text (city, country) in the visible label.
- Hardened profile website links to always include noopener noreferrer with target="_blank" while preserving nofollow/ugc behavior logic.
- Bumped the plugin version in plugin.rb from 7.1.7 to 7.1.8.
Highlights include:
you can now update people’s location automatically upon login (not just when they post) (@祁同伟 )
controlled by new setting DEFAULT OFF - so if you use this feature, set your preference now or Locations will stop updating users locations completely - necessary as style of setting was changed to accommodate the new feature.
Map can now once again be selected as default view for a Category (via Category Settings) (@Quacken )
Topic default location from user location works again. (@祁同伟 )
Plus a thousand modernisations to latest linting rules.
7 Likes