search

apigee / apigee-edge-drupal

The Apigee Edge module enables you to integrate a Drupal 9 or 8 site with Apigee.
https://www.drupal.org/project/apigee_edge
GNU General Public License v2.0
32 stars 45 forks source link

Edit AppGroup is clearing the previously added developers in the appgroup #1074

Closed sindhuputti closed 3 months ago

sindhuputti commented 3 months ago

Description

Create an appgroup, add members to the appgroup. Edit AppGroup, it is clearing the previously set developers in the appgroup.

Steps to Reproduce

Steps to reproduce the behavior:

  1. Add a new Appgroup on developer portal and add some members to it.
  2. Edit the appgroup, it will reset all the existing members, only one developer who is editing the appgroup is being added as a developer.

Actual Behavior

When editing the appgroup, all the previously added members after the appgroup creation are gone.

Expected Behavior

An appgroup should persist the previously added developers.

Screenshots

If applicable, add screenshots to help explain your problem.

Notes

Add any other context about the problem here.

Version Info

This can be the version you can see on admin/modules in Drupal or the output of this command: composer show. Add Drupal core and other version information if needed. apigee/apigee-client-php 3.0.5 Client library for connecting to the Apigee Edge API. apigee/apigee-mock-client-php 1.1.2 To be used for queueing mock API responses for the apigee/apigee-client-php library. apigee/apigee_devportal_kickstart 2.1.6 A fast demo and starting point for Apigee Developer Portals. drupal/core 10.1.8 Drupal is an open source content management platform powering millions of websites and applications. drupal/core-composer-scaffold 10.1.8 A flexible Composer project scaffold builder. drupal/core-project-message 10.1.8 Adds a message after Composer installation. drupal/core-recommended 10.1.8 Core and its dependencies with known-compatible minor versions. Require this project INSTEAD OF drupal/core.

kedarkhaire commented 3 months ago

Hi @sindhuputti I will check and look into it. Thanks!

kedarkhaire commented 3 months ago

Hi @sindhuputti We checked the above implementation, Post editing the Appgroup, no members were deleted unless the Team member sync was ran. We saw post running the Team member sync only when the Appgroup are edited/updated, we see the missing members on Appgroup. We assume that there might be some code in your other custom module to sync the team members on AppGroup update, which leads to this issue. Anyways this should not happen, so we have created a PR & patch for it.

Please try to apply this patch and post your checks do let us know, if it works for you.

Thanks!