mappjzc

Results 8 comments of mappjzc

feat: embeddable config-ui

thanks for your pr. @e2corporation can you help for review it?

feat: embeddable config-ui

> @mappjzc This seems like a separate UI project, why is it bundled with `incubator-devlake` project and not a new standalone project? @e2corporation This is not my submitted pr. Maybe...

[why.md][documentation] why.md

Thank you for your suggestions. @Startrekzky can you help with this issue?

[Bug][github] some /pulls/comments of repo get servers error

@keon94 In config ui, we may need more friendly error messages.

[Feature][Jira] Allow users to define their own standard issue types and statuses

document part dismantling disassembled to #2864

[Refactor][gitlab,jira,jenkins,tapd] Missing error handling

Thanks for reporting the issue. I'm looking forward to your PR.

[Refactor][gitlab,jira,jenkins,tapd] Missing error handling

![image](https://user-images.githubusercontent.com/2921251/180344964-5c5ea14c-dd23-4903-a8cb-b2647c781cfc.png) I have finished to looking forward to your PR. The err has been return whether it is empty or not. You can changed the code as ``` if err...

perf: added error handling missing code

![image](https://user-images.githubusercontent.com/2921251/180343690-63e89f7b-d06b-475d-900b-d21bb15f83ff.png) Whether error is empty or not, error is returned. So we don't need to make "err ! = nil" judgment before this.