Problem/Motivation

  • Fix the error that Key pair name is displayed as "undefined" in the SPA.

Issue fork cloud-3362829

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:

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Ryo Yamashita created an issue. See original summary.

Ryo Yamashita’s picture

Status: Needs work » Needs review

@yas

Please review it. Thanks.

yas’s picture

Title: Fix the error that Key pair name is displayed as "undefined" in the SPA » Fix an error that OpenStack key pair name is displayed as "undefined" in the SPA
Status: Needs review » Needs work
Issue tags: +chatgpt

@ryo-yamashita

Thank you for the patch. I left my comment. Thanks!

Ryo Yamashita’s picture

Status: Needs work » Needs review

@yas

I fixed the patch for your comments. Thanks.

yas’s picture

Title: Fix an error that OpenStack key pair name is displayed as "undefined" in the SPA » Fix an issue where the OpenStack key pair name was displayed as 'undefined' in the SPA
Category: Task » Bug report
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

@ryo-yamashita

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

  • yas committed 987efebd on 6.x authored by Ryo Yamashita
    Issue #3362829 by Ryo Yamashita, yas: Fix an issue where the OpenStack...

  • yas committed d1146836 on 5.x authored by Ryo Yamashita
    Issue #3362829 by Ryo Yamashita, yas: Fix an issue where the OpenStack...
yas’s picture

Status: Reviewed & tested by the community » Needs work

@ryo-yamashita

Sorry, I've merged the patch w/ some error: https://www.drupal.org/pift-ci-job/2676588. Can you please create the hotfix for drupal/cloud6.x-dev?

I guess that it is related to the following:

 .../cloud_dashboard/src/organisms/PageTitle.tsx    |   16 +++++-
 modules/cloud_dashboard/js/index.js                |   51 +++++++++++---------
 2 files changed, 42 insertions(+), 25 deletions(-)

1801.diff:41: trailing whitespace.
(function(){const t=document.createElement("link").relList;if(t&&t.supports&&t.supports("modulepreload"))return;for(const i of document.querySelectorAll('link[rel="modulepreload"]'))r(i);new MutationObserver(i=>{for(const o of i)if(o.type==="childList")for(const u of 

... (snip) ...

Week");Drupal.t("1 Year");Drupal.t("1 day");Drupal.t("1 item");Drupal.t("1 week");Drupal.t("1");Drupal.t(`16-32 Characters, e.g. 12ABCDEFGHIJKVWXYZ89 .  If <em>Assume Role</em> 
1801.diff:50: trailing whitespace.
revision.`);Drupal.t("Info");Drupal.t("Installed/Updated the entity type(s): @entity_type_ids");Drupal.t("Instance ID");Drupal.t("Instance Link");Drupal.t("Instance Name");Drupal.t("Instance Pricing");Drupal.t("Instance Shutdown Behavior");Drupal.t("Instance State");Drupal.t("Instance 

... (snip) ...

quota.");Drupal.t("Message");Drupal.t("Metric");Drupal.t("Metrics");Drupal.t("Mi");Drupal.t("Min");Drupal.t("Minute");Drupal.t("Module");Drupal.t("Monday");Drupal.t("Monitor");Drupal.t("Monitoring");Drupal.t("Monthly");Drupal.t(`More than one elastic IP can appear in the email message. Available 
warning: 2 lines add whitespace errors.
yas’s picture

Status: Needs work » Fixed

@ryo-yamashita

Let us close this issue since the coding standard errors should be handled in the other issue.

Thanks

Status: Fixed » Closed (fixed)

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