Problem/Motivation

  • Migrate to native JavaScript from jQuery (1) (cloud_geocoder.js)

Issue fork cloud-3399268

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

Masami created an issue. See original summary.

Masami opened merge request !2084

masami’s picture

masami commented
Status: Active » Needs review
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Issue summary: View changes
Status: Needs review » Needs work

@masami

Thank you for the patch. I posted my minor comment. Thanks!

masami’s picture

masami commented
Status: Needs work » Needs review
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Title: Migrate to native JavaScript from jQuery (1) (cloud_geocoder.js) » Migrate to native JavaScript from jQuery (cloud_geocoder.js)
Status: Needs review » Reviewed & tested by the community

@masami

Thank you for the update. It looks good. I'll merge the patch to 5.x and 6.x, and close this issue as Fixed.

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Title: Migrate to native JavaScript from jQuery (cloud_geocoder.js) » Refactor to native JavaScript from jQuery (cloud_geocoder.js)

  • yas committed 3f843d4d on 6.x authored by Masami 
    Issue #3399268 by Masami, yas: Refactor to native JavaScript from jQuery...

  • yas committed 5b6bde34 on 5.x authored by Masami 
    Issue #3399268 by Masami, yas: Refactor to native JavaScript from jQuery...
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Status: Reviewed & tested by the community » Fixed
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Issue tags: +chatgpt

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.