merefield
Fevereiro 26, 2026, 2:29pm
1224
Versão de manutenção:
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.
seguida por uma correção:
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.
Os destaques incluem:
agora você pode atualizar a localização das pessoas automaticamente no login (não apenas quando elas postam) (@祁同伟 )
controlado pela nova configuração PADRÃO DESLIGADO - então, se você usar este recurso, defina sua preferência agora ou Localizações parará de atualizar completamente as localizações dos usuários - necessário, pois o estilo da configuração foi alterado para acomodar o novo recurso.
O mapa pode ser selecionado novamente como visualização padrão para uma Categoria (via Configurações da Categoria) (@Quacken )
A localização padrão do Tópico a partir da localização do usuário funciona novamente. (@祁同伟 )
Além de mil modernizações para as regras de linting mais recentes.
7 curtidas