Refactor to native JavaScript from jQuery (aws_cloud_instance_monitor_horizon_chart.js)

Issue created by @Masami
Comment 8 months ago →
🇯🇵Japan Masami
Merge request !2085Issue #3399270 by Masami, yas: Migrate to native JavaScript from jQuery (8)... → (Closed) created by Masami
Open in Jenkins → Open on Drupal.org →
Core: 10.1.4 + Environment: PHP 8.2 & MySQL 8
last update 8 months ago
Composer require failure
Status changed to Needs review 8 months ago2:17am 8 November 2023
Comment 8 months ago →
🇯🇵Japan Masami
First commit to issue fork.
Open in Jenkins → Open on Drupal.org →
Core: 10.1.4 + Environment: PHP 8.2 & MySQL 8
last update 8 months ago
Composer require failure
Status changed to Needs work 8 months ago2:20am 8 November 2023
Comment 8 months ago →
🇺🇸United States yas California 🇺🇸
@masami

Thank you for the patch. I posted my minor comment. Thanks!
Open in Jenkins → Open on Drupal.org →
Core: 10.1.4 + Environment: PHP 8.2 & MySQL 8
last update 8 months ago
Composer require failure
Open in Jenkins → Open on Drupal.org →
Core: 10.1.4 + Environment: PHP 8.2 & MySQL 8
last update 8 months ago
Composer require failure
Open in Jenkins → Open on Drupal.org →
Core: 10.1.4 + Environment: PHP 8.2 & MySQL 8
last update 8 months ago
Composer require failure
Open in Jenkins → Open on Drupal.org →
Core: 10.1.4 + Environment: PHP 8.2 & MySQL 8
last update 8 months ago
Composer require failure
Status changed to Needs review 8 months ago6:43am 12 November 2023
Comment 8 months ago →
🇯🇵Japan Masami
Comment 8 months ago →
🇺🇸United States yas California 🇺🇸
Status changed to Needs work 8 months ago7:20am 12 November 2023
Comment 8 months ago →
🇺🇸United States yas California 🇺🇸
@masami

It looks there is a redundant blank line. Can you please fix it? Thanks
Open in Jenkins → Open on Drupal.org →
Core: 10.1.4 + Environment: PHP 8.2 & MySQL 8
last update 8 months ago
Composer require failure
Status changed to Needs review 8 months ago7:34am 12 November 2023
Comment 8 months ago →
🇯🇵Japan Masami
Status changed to Needs work 8 months ago7:49am 12 November 2023
Comment 8 months ago →
🇺🇸United States yas California 🇺🇸
@masami

Thank you for the update. Please rebase the patch. Thanks
Comment 8 months ago →
🇺🇸United States yas California 🇺🇸
Open on Drupal.org →
Core: 10.1.4 + Environment: PHP 8.2 & MySQL 8
last update 7 months ago
Not currently mergeable.
Open in Jenkins → Open on Drupal.org →
Core: 10.1.4 + Environment: PHP 8.2 & MySQL 8
last update 7 months ago
Composer require failure
Open in Jenkins → Open on Drupal.org →
Core: 10.1.4 + Environment: PHP 8.2 & MySQL 8
last update 7 months ago
Composer require failure
Open in Jenkins → Open on Drupal.org →
Core: 10.1.4 + Environment: PHP 8.2 & MySQL 8
last update 7 months ago
Composer require failure
Status changed to Needs review 7 months ago12:09am 4 December 2023
Comment 7 months ago →
🇯🇵Japan Masami
Status changed to Needs work 7 months ago9:19am 4 December 2023
Comment 7 months ago →
🇺🇸United States yas California 🇺🇸
@masami

Thank you for the refactoring. I posted my comments. Thanks!
Open in Jenkins → Open on Drupal.org →
Core: 10.1.4 + Environment: PHP 8.2 & MySQL 8
last update 7 months ago
Composer require failure
Comment 7 months ago →
🇯🇵Japan Masami
Masami → changed the visibility of the branch 3399270-migrate-native-js to hidden.
Merge request !2137Issue #3399270 by Masami, yas: Refactor to native JavaScript from jQuery... → (Merged) created by Masami
Open in Jenkins → Open on Drupal.org →
Core: 10.1.4 + Environment: PHP 8.2 & MySQL 8
last update 7 months ago
Composer require failure
Comment 7 months ago →
System Message
Masami → closed merge request !2085
Status changed to Needs review 7 months ago11:18am 9 December 2023
Comment 7 months ago →
🇯🇵Japan Masami
Status changed to Needs work 7 months ago2:41pm 10 December 2023
Comment 7 months ago →
🇺🇸United States yas California 🇺🇸
@masami

Thank you for the update. I posted my additional comment. Can you please take a look at it? Thanks!
Open in Jenkins → Open on Drupal.org →
Core: 10.1.4 + Environment: PHP 8.2 & MySQL 8
last update 7 months ago
Composer require failure
Open in Jenkins → Open on Drupal.org →
Core: 10.1.4 + Environment: PHP 8.2 & MySQL 8
last update 7 months ago
Composer require failure
Open in Jenkins → Open on Drupal.org →
Core: 10.1.4 + Environment: PHP 8.2 & MySQL 8
last update 7 months ago
Composer require failure
Status changed to Needs review 7 months ago5:52am 16 December 2023
Comment 7 months ago →
🇯🇵Japan Masami
Open in Jenkins → Open on Drupal.org →
Core: 10.1.4 + Environment: PHP 8.2 & MySQL 8
last update 7 months ago
Composer require failure
Status changed to RTBC 7 months ago2:58pm 18 December 2023
Comment 7 months ago →
🇺🇸United States yas California 🇺🇸
@masami

Thank you for the update. I'll merge the patch to 5.x and 6.x, and close this issue as Fixed.
Open in Jenkins → Open on Drupal.org →
Core: 10.1.4 + Environment: PHP 8.2 & MySQL 8
last update 7 months ago
Composer require failure
Comment 7 months ago →
System Message

yas → committed ebd720f8 on 6.x authored by Masami →
Issue #3399270 by Masami, yas: Refactor to native JavaScript from jQuery...
Comment 7 months ago →
System Message

yas → committed 16a4f69e on 5.x authored by Masami →
Issue #3399270 by Masami, yas: Refactor to native JavaScript from jQuery...
Status changed to Fixed 7 months ago3:29pm 18 December 2023
Comment 7 months ago →
🇺🇸United States yas California 🇺🇸
Comment 6 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

Refactor to native JavaScript from jQuery (aws_cloud_instance_monitor_horizon_chart.js)

Problem/Motivation

Merge Requests

!2137Refactor to native JavaScript from jQuery (aws_cloud_instance_monitor_horizon_chart.js)
Merged

!2085Refactor to native JavaScript from jQuery (aws_cloud_instance_monitor_horizon_chart.js)
Closed

Comments & Activities

Refactor to native JavaScript from jQuery (aws_cloud_instance_monitor_horizon_chart.js)

Problem/Motivation

Merge Requests

!2137Refactor to native JavaScript from jQuery (aws_cloud_instance_monitor_horizon_chart.js)Merged

!2085Refactor to native JavaScript from jQuery (aws_cloud_instance_monitor_horizon_chart.js)Closed

Comments & Activities

!2137Refactor to native JavaScript from jQuery (aws_cloud_instance_monitor_horizon_chart.js)
Merged

!2085Refactor to native JavaScript from jQuery (aws_cloud_instance_monitor_horizon_chart.js)
Closed