klesh opened a new issue, #5371:
URL: https://github.com/apache/incubator-devlake/issues/5371

   ## What and why to refactor
   Previously, we would load the `scopeConfigName` for `list scopes` and `get 
scope` APIs for all our plugins, so that config-ui could be able to display the 
related `scopeConfig` without calling another API. However, there are a couple 
of complication:
   
   1. The new prototype requires `DataEntities` to be displayed when listing 
scopes or viewing a specific scope.
   2. with #5159, the DataEntities option was moved into ScopeConfig, which 
lead to an extra API request.
   
   ## Describe the solution you'd like
   Most of us if not all had came to an agreement that those APIs should 
provide `ScopeConfig` entity for simplicity's sake.
   
   1. Remove `scopeConfigName` from the API response
   2. Add `scopeConfig` which essentially is the related scope config entity if 
`scope_config_id` is available
   
   ## Related issues
   #5159 
   
   
   @keon94 @CamilleTeruel python plugins need to adopt as well.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to