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

   ## Question
   Hello,
   
   I have noticed that a new plugin called `trello` has been merged into the 
main branch. I would like to thank @jakezhu9  for his contributions to this 
plugin. However, I have identified some minor issues with it.
   
   Firstly, the migration script refers to live models which can evolve over 
time, making the effects of the migration script uncertain. To address this 
issue, we should archive the models to a specified package, just as we have 
done with other plugins. This way, the migration scripts will only refer to the 
models in this package.
   
   Secondly, two new helpers, `ScopeAPIHelper` and `TransformationRuleHelper`, 
have been introduced to the 
`github.com/apache/incubator-devlake/helpers/pluginhelper/api package`. It 
would be better to adopt these helpers in the HTTP handlers for scope and 
transformation rules.
   
   Thank you for your attention to these matters. I believe that these changes 
will help to improve the stability and usability of the `trello` plugin.
   
   


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