- Issue created by @Masami
- Merge request !2107Issue #3402646 by Masami, yas: Migrate to native JavaScript from jQuery (11)... → (Merged) created by Masami
- last update
about 1 year ago Composer require failure - Status changed to Needs review
about 1 year ago 8:37am 19 November 2023 - Status changed to Needs work
about 1 year ago 5:14am 20 November 2023 - 🇺🇸United States yas California 🇺🇸
@masami
Thank you for the refactoring. I posted my comments. Thanks
- First commit to issue fork.
- last update
about 1 year ago Composer require failure - Status changed to Needs review
about 1 year ago 9:13am 20 November 2023 - 🇮🇳India ankithashetty Karnataka, India
Hello @masami, @yas, addressed the minor feedback from #4 and updated the MR accordingly.
Thanks!
- 🇺🇸United States yas California 🇺🇸
@ankithashetty
Thank you for your fixes. It helped us. However unfortunately I've missed the other places. It is appreciated if you can take a look at the ones. Thanks!
- Status changed to Needs work
about 1 year ago 9:33am 20 November 2023 - last update
about 1 year ago Composer require failure - Status changed to Needs review
about 1 year ago 9:55am 20 November 2023 - 🇮🇳India ankithashetty Karnataka, India
Addressed the feedbacks, thanks again!
- 🇺🇸United States yas California 🇺🇸
@ankithashetty
I'll merge the patch to
5.x
and6.x
, and close this issue as Fixed. - last update
about 1 year ago Composer require failure -
yas →
committed ac08ddea on 6.x authored by
Masami →
Issue #3402646 by Masami, ankithashetty, yas: Refactor to native...
-
yas →
committed ac08ddea on 6.x authored by
Masami →
-
yas →
committed c6058873 on 5.x authored by
ankithashetty →
Issue #3402646 by Masami, ankithashetty, yas: Refactor to native...
-
yas →
committed c6058873 on 5.x authored by
ankithashetty →
- Status changed to Fixed
about 1 year ago 9:05pm 20 November 2023 Automatically closed - issue fixed for 2 weeks with no activity.