Locations Plugin 🌍

[merefield (Robert)]

FIX: Map route issues (#90)

• 수정: 전역(Topic) 지도를 직접 열 때 문제가 발생하지 않도록 수정
• 수정: 카테고리 지도와 전역 지도 간 전환이 이제 올바르게 작동하도록 수정
• 작동하지 않는 WIP 지도 테스트

@vincefrommtl, @Stephane_Roy @packman 이 업데이트를 테스트해 주시겠습니까? 수정 범위를 참고해 주세요. 곧 추가 수정 사항이 올 예정입니다.

(기술적 참고: Leaflet 라이브러리가 전역 객체이기 때문에 QUnit에서 이를 인식하지 못하는 것 같습니다. 따라서 이 수정 사항에 대해 완전히 통합된 FE 테스트를 제공할 수 없었습니다. Leaflet과 QUnit을 함께 사용하게 하는 방법을 아시는 분이라면 알려 주시기 바랍니다.)

[vincefrommtl]

피드백:

• 전역 맵과 카테고리 맵 간의 전환은 정상적으로 작동합니다. 하지만 반대 방향(카테고리 맵에서 전역 맵으로)은 카테고리 간 전환과 마찬가지로 깨져 있습니다.
• 업그레이드 이후 로그에서 이 오류를 확인하지 못했습니다: ActionView::Template::Error (No route matches {:action=>"map_feed", :controller=>"list"}).
• 새로운 버그: 때때로 맵에서 클러스터링된 링크가 확대했을 때 위치 마커만 표시되고 라벨은 표시되지 않습니다.
• 새로운 버그: 때때로 맵의 마커 라벨이 해당 주제에 링크되지 않습니다.
• 새로운 버그: 전역 맵이 이제 카테고리 맵과 동일한 버그의 영향을 받고 있습니다. 즉, 일부 마커만 표시됩니다(아래 참조).

마커 수가 예상보다 적은 맵에 대해: 자세히 살펴본 결과, 이는 최근 20일(또는 그 정도) 동안 활동이 있었던 주제에 대한 마커만 표시하고 있으며, 최대 30개까지 표시됩니다. 이는 우리 포럼의 설정 문제일 수 있을까요? 매우 구체적인 설정처럼 보입니다! 하지만 우리는 해당 설정에서 아무것도 변경하지 않았습니다. 위치 맵 최대 주제 수는 최대 1700으로 설정되어 있으며, 최근 20일 동안 활동이 있었던 주제의 마커만 표시하도록 하는 시간 관련 설정이 있는지 의문입니다.

[packman (Chris McMahon)]

업그레이드 후 글로벌 맵을 열었는데 마커(토픽)가 표시되지 않았습니다. 마커가 있는 토픽의 위치를 편집한 뒤 변경 사항 없이 그냥 저장하면 마커가 나타납니다. 제 마커는 모두 20일 이상 된 것이므로, 아마도 @vincefrommtl 님이 보고 있는 문제와 같은 것일까요?

수정: 각 토픽의 ‘맵 보기’ 버튼은 글로벌 맵에 마커가 표시되지 않더라도 올바른 마커 위치를 표시합니다.

[merefield (Robert)]

감사합니다, 여러분. 한 번 더 확인해 볼게요.

[merefield (Robert)]

이것은 완전한 수정은 아니지만, 제 생각에는 외부 링크에서 직접 로드하거나 브라우저를 새로고침할 때 발생하는 맵 오류를 피하면서 이전의 주제 위치를 복원할 것입니다:

FIX: old topics not included on Map (#91)

알려진 문제: 직접적인 Map => Category Map 전환은 해결하지 못했습니다(하지만 이는 경계 사례여야 합니다). 그러나 그 외 모든 사항은 개선되어야 합니다.

[packman (Chris McMahon)]

내일 테스트할 예정인데, 방금 한 사용자가 미국 지역이 이제 ‘도시, 국가, 주’ 순서로 표기되고 있다고 지적했습니다. 예를 들어, Atlanta, US, Georgia처럼요.

[vincefrommtl]

간단한 테스트 결과, 이 새로운 업데이트를 통해 지도의 모든 마커를 볼 수 있음을 확인했습니다. 정말 감사합니다.

당분간은 사용자가 하나 이상의 카테고리 맵으로 이동할 때 깨진 전환을 우회하도록 조언할 예정입니다.

[merefield (Robert)]

최근 변경 사항은 모두 맵 인구와 관련이 있습니다.

[packman (Chris McMahon)]

국가/주 순서가 최신 버전이 아닙니다. 지난 며칠간의 버전에서 이를 본 적이 없다고 기억하기 때문에 언급한 것입니다.

[merefield (Robert)]

@packman 지금까지는 이전에 발생한 문제를 수정했습니다:

FIX: location default when geolocation is empty (#89)

이제 테스트로 커버되고 있습니다.

해당 문제를 지하실에서 꺼내준 @david에게 다시 한번 감사드립니다.

[blake (Blake Erickson)]

@merefield 안녕하세요, 간단히 말씀드리면 modify_user_params 메서드는 이미 오래전부터 비추천(deprecated) 상태이며, Discourse가 3.2.0.beta1-dev 버전으로 업데이트되었기 때문에 곧 제거될 예정입니다.

discourse-locations/lib/users_map.rb
35:  def modify_user_params(attrs)

시간 나실 때 users_controller_update_user_params 수정자(modifier)를 사용하도록 업데이트해 주실 수 있을까요? 감사합니다!

[merefield (Robert)]

네, 추가 정보를 알려주셔서 감사합니다. 이번 주에 확인해 보겠습니다.

[merefield (Robert)]

Blake, 작업이 완료되었는데 리뷰어로 추가해도 될까요?

COMPATIBILTIY: resolve deprecation of controller method (#92)

[blake (Blake Erickson)]

멋지네요, 감사합니다!

[merefield (Robert)]

감사합니다, 병합했습니다!

[Richie (Richie Rich)]

나쁜 소식을 전하게 되어 죄송합니다. 버그 보고가 있습니다.

현재 최신 버전을 사용 중이며, 오늘 아침 업데이트했습니다: 3.2.0.beta1-dev (7ca5ee6cd2)

2주 전 마지막 업데이트 시에는 없었던 이상한 문제가 발생하여, 사용자 정의 필드가 있는 사용자의 프로필을 더 이상 편집할 수 없습니다.

다음 사용자를 편집하려고 합니다: https://example.com/u/username/preferences/profile

페이지는 정상적으로 로드됩니다:

저장을 클릭하는 즉시 오류가 발생합니다:

이 문제가 드러난 이유는 우리 클럽 멤버가 등급을 업그레이드할 때 다른 웹사이트 중 하나가 “membership” 사용자 정의 필드를 업데이트하려고 시도하기 때문인데, 이때 Discourse API 엔드포인트가 500 오류를 발생시키고 있습니다.

Postman으로 재현을 시도해도 동일한 문제가 나타납니다:

Discourse 로그에는 다음과 같은 내용이 표시됩니다:

그리고:

plugins/discourse-locations/plugin.rb:188:in `block (2 levels) in activate!'

lib/discourse_plugin_registry.rb:293:in `apply_modifier'

app/controllers/users_controller.rb:2036:in `user_params'

app/controllers/users_controller.rb:198:in `update'

actionpack (7.0.7) lib/action_controller/metal/basic_implicit_render.rb:6:in `send_action'

actionpack (7.0.7) lib/abstract_controller/base.rb:215:in `process_action'

actionpack (7.0.7) lib/action_controller/metal/rendering.rb:165:in `process_action'

actionpack (7.0.7) lib/abstract_controller/callbacks.rb:234:in `block in process_action'

activesupport (7.0.7) lib/active_support/callbacks.rb:118:in `block in run_callbacks'

app/controllers/application_controller.rb:420:in `block in with_resolved_locale'

i18n (1.14.1) lib/i18n.rb:322:in `with_locale'

app/controllers/application_controller.rb:420:in `with_resolved_locale'

activesupport (7.0.7) lib/active_support/callbacks.rb:127:in `block in run_callbacks'

activesupport (7.0.7) lib/active_support/callbacks.rb:138:in `run_callbacks'

actionpack (7.0.7) lib/abstract_controller/callbacks.rb:233:in `process_action'

actionpack (7.0.7) lib/action_controller/metal/rescue.rb:23:in `process_action'

actionpack (7.0.7) lib/action_controller/metal/instrumentation.rb:67:in `block in process_action'

activesupport (7.0.7) lib/active_support/notifications.rb:206:in `block in instrument'

activesupport (7.0.7) lib/active_support/notifications/instrumenter.rb:24:in `instrument'

activesupport (7.0.7) lib/active_support/notifications.rb:206:in `instrument'

actionpack (7.0.7) lib/action_controller/metal/instrumentation.rb:66:in `process_action'

actionpack (7.0.7) lib/action_controller/metal/params_wrapper.rb:259:in `process_action'

activerecord (7.0.7) lib/active_record/railties/controller_runtime.rb:27:in `process_action'

actionpack (7.0.7) lib/abstract_controller/base.rb:151:in `process'

actionview (7.0.7) lib/action_view/rendering.rb:39:in `process'

actionpack (7.0.7) lib/action_controller/metal.rb:188:in `dispatch'

actionpack (7.0.7) lib/action_controller/metal.rb:251:in `dispatch'

actionpack (7.0.7) lib/action_dispatch/routing/route_set.rb:49:in `dispatch'

actionpack (7.0.7) lib/action_dispatch/routing/route_set.rb:32:in `serve'

actionpack (7.0.7) lib/action_dispatch/journey/router.rb:50:in `block in serve'

actionpack (7.0.7) lib/action_dispatch/journey/router.rb:32:in `each'

actionpack (7.0.7) lib/action_dispatch/journey/router.rb:32:in `serve'

actionpack (7.0.7) lib/action_dispatch/routing/route_set.rb:852:in `call'

lib/middleware/omniauth_bypass_middleware.rb:74:in `call'

rack (2.2.8) lib/rack/tempfile_reaper.rb:15:in `call'

rack (2.2.8) lib/rack/conditional_get.rb:40:in `call'

rack (2.2.8) lib/rack/head.rb:12:in `call'

actionpack (7.0.7) lib/action_dispatch/http/permissions_policy.rb:38:in `call'

lib/content_security_policy/middleware.rb:12:in `call'

lib/middleware/anonymous_cache.rb:389:in `call'

lib/middleware/gtm_script_nonce_injector.rb:10:in `call'

rack (2.2.8) lib/rack/session/abstract/id.rb:266:in `context'

rack (2.2.8) lib/rack/session/abstract/id.rb:260:in `call'

actionpack (7.0.7) lib/action_dispatch/middleware/cookies.rb:704:in `call'

actionpack (7.0.7) lib/action_dispatch/middleware/callbacks.rb:27:in `block in call'

activesupport (7.0.7) lib/active_support/callbacks.rb:99:in `run_callbacks'

actionpack (7.0.7) lib/action_dispatch/middleware/callbacks.rb:26:in `call'

actionpack (7.0.7) lib/action_dispatch/middleware/debug_exceptions.rb:28:in `call'

actionpack (7.0.7) lib/action_dispatch/middleware/show_exceptions.rb:29:in `call'

logster (2.12.2) lib/logster/middleware/reporter.rb:43:in `call'

railties (7.0.7) lib/rails/rack/logger.rb:40:in `call_app'

railties (7.0.7) lib/rails/rack/logger.rb:27:in `call'

config/initializers/100-quiet_logger.rb:20:in `call'

config/initializers/100-silence_logger.rb:29:in `call'

actionpack (7.0.7) lib/action_dispatch/middleware/remote_ip.rb:93:in `call'

actionpack (7.0.7) lib/action_dispatch/middleware/request_id.rb:26:in `call'

lib/middleware/enforce_hostname.rb:24:in `call'

rack (2.2.8) lib/rack/method_override.rb:24:in `call'

actionpack (7.0.7) lib/action_dispatch/middleware/executor.rb:14:in `call'

rack (2.2.8) lib/rack/sendfile.rb:110:in `call'

actionpack (7.0.7) lib/action_dispatch/middleware/host_authorization.rb:131:in `call'

message_bus (4.3.7) lib/message_bus/rack/middleware.rb:60:in `call'

lib/middleware/request_tracker.rb:233:in `call'

railties (7.0.7) lib/rails/engine.rb:530:in `call'

railties (7.0.7) lib/rails/railtie.rb:226:in `public_send'

railties (7.0.7) lib/rails/railtie.rb:226:in `method_missing'

rack (2.2.8) lib/rack/urlmap.rb:74:in `block in call'

rack (2.2.8) lib/rack/urlmap.rb:58:in `each'

rack (2.2.8) lib/rack/urlmap.rb:58:in `call'

unicorn (6.1.0) lib/unicorn/http_server.rb:634:in `process_client'

unicorn (6.1.0) lib/unicorn/http_server.rb:739:in `worker_loop'

unicorn (6.1.0) lib/unicorn/http_server.rb:547:in `spawn_missing_workers'

unicorn (6.1.0) lib/unicorn/http_server.rb:143:in `start'

unicorn (6.1.0) bin/unicorn:128:in `<top (required)>'

vendor/bundle/ruby/3.2.0/bin/unicorn:25:in `load'

vendor/bundle/ruby/3.2.0/bin/unicorn:25:in `<main>'

이상하게도, 제 자신의 프로필은 사용자 정의 필드를 포함하여 정상적으로 편집할 수 있습니다.

하지만 다른 사용자는 더 이상 편집할 수 없으며, system 계정도 우리의 API를 통해 업데이트를 수행할 수 없습니다.

Locations 플러그인을 비활성화하면 모든 것이 다시 정상적으로 작동합니다:

[merefield (Robert)]

@Richie 보고해 주셔서 감사합니다.

재현해 보았습니다.

누군가가 위치 정보가 없는 경우에만 발생하는 것 같습니다.

코어에서 비추천(deprecation) 처리된 항목으로 인해 최근 리팩토링을 했는데, (꽤 중요한) 케이스를 놓치고 있었습니다.

곧 패치하겠습니다.

[Richie (Richie Rich)]

@merefield

위치 정보가 없는 경우에만 발생하는 것 같습니다.

아, 그러면 제가 제 프로필을 성공적으로 편집한 건 우연이었을 수도 있겠네요. 제 프로필에는 위치 정보가 있으니까요

@merefield

곧 패치하겠습니다

걱정 마세요, 계속 도와주셔서 감사합니다

[merefield (Robert)]

지금 수정되었을 것입니다:

FIX: user without location changing user field should not error (#94)

[Richie (Richie Rich)]

@merefield

이제 수정되었을 것 같습니다.

오늘 아침 업데이트 결과, 더 이상 HTTP/500 오류가 발생하지 않습니다.

빠르게 수정해 주셔서 감사합니다, Robert.