keon94 commented on code in PR #5506:
URL: 
https://github.com/apache/incubator-devlake/pull/5506#discussion_r1232502068


##########
backend/helpers/pluginhelper/api/connection_helper.go:
##########
@@ -101,8 +104,16 @@ func (c *ConnectionApiHelper) List(connections 
interface{}) errors.Error {
 }
 
 // Delete connection
-func (c *ConnectionApiHelper) Delete(connection interface{}) errors.Error {
-       return CallDB(c.db.Delete, connection)
+func (c *ConnectionApiHelper) Delete(plugin string, connection interface{}) 
(*services.BlueprintProjectPairs, errors.Error) {

Review Comment:
   It can be done, we just have to go and hardcode constants (the name) for 
each plugin. Right now, the plugins are registered by the names derived from 
their compiled *.so files. So as long as there is never a discrepancy between 
those names and these constants, it'll be ok.



-- 
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