mappjzc commented on PR #4625: URL: https://github.com/apache/incubator-devlake/pull/4625#issuecomment-1471983826
> @jakezhu9 @mappjzc I see, I think it would be better to use the `AppKey` instead of the `BasicAuth`. Note that we can modify the `req.Query` in the `SetupAuthentication` method as well to avoid confusion regarding which authorization method we opted for. Keep in mind we should add a document to https://devlake.apache.org/ at the end of the development to explain the authorization etc. Thanks for your hard work, appreciated it. 💪 Thanks for your help. I had already succeeded to test your connection. It works. And I try to run a blueprint for it, But here have some problems.  As you see, It sends out the error that "boardId is required". This error comes from PrepareTaskData as follows.  It decodes from the map option to TrelloOptions.  And in TrelloOptions, It required BoardId with json field "boardId" But in the blueprint, there is only filed "scopeId" in the JSON string.  To see the makePipelinePlanV200 of your code, you use the filed "scopeId" in the options JSON string.  Maybe you need to unify them in the way you like. After I fixed this on my locally, this plugin can work successfully. And another little thing. the CollectCheckItem and CollectChecklist Collect the same data. It is enough only one. -- 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]
