Problem/Motivation

  • If the Cloud module is installed, for users with the role of Cloud administrator, display a link to the user management page from the Manage menu of Cloud Orchestrator in SPA.
  • Add a link to Users (/admin/people) under Manage in the main menu when you installed the Cloud module.
  • Only users who have permission to view the user management screen will see the link.

Issue fork cloud-3363113

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

hosomitm created an issue. See original summary.

hosomitm opened merge request !1803

hosomitm’s picture

hosomitm commented
Status: Active » Needs review

@yas @ryo-yamashita

Please review the code.

The "Administration" link is displayed when logged in as a user with the View cloud service provider administration list permission.
The "Users" link is displayed when logged in as a user with the Administer users permission and the RoleAssign module is installed.

I have added an API to the cloud module's ApiController for determining whether each menu should be displayed. Is there any issue with adding it in that location?

Thank you.

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented

@hosomitm

Thank you for the patch. It looks good to me.

@ryo-yamashita

What do you think?

Thanks

ryo yamashita’s picture

ryo yamashita commented
Status: Needs review » Reviewed & tested by the community

@yas @hosomitm

it looks good to me. Thanks.

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented

@ryo-yamashita

Thank you for your review.

@hostomitm

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

  • yas committed 4084fea5 on 6.x authored by hosomitm 
    Issue #3363113 by hosomitm, yas, Ryo Yamashita: Add a link to display...

  • yas committed e4d93150 on 5.x authored by hosomitm 
    Issue #3363113 by hosomitm, yas, Ryo Yamashita: Add a link to display...
yas’s picture

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

Status: Fixed » Closed (fixed)

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