[Feature][Config UI] About jenkins connections modularity.
Search before asking
- [X] I had searched in the issues and found no similar feature requirement.
Description
In my work, I need to constantly add new functions on the Config UI to our system, which is very painful for me, so I hope to be able to directly reference the Config UI to alleviate this repetitive work.
After some discussions today, we found that making the Config UI embeddable is a complicated thing, and it cannot be done in a short time, so we hope to use the Jenkins connection as an example to try to make This part becomes embeddable.
So I created this issue to try to do this.
Use case
Make Jenkins connection embeddable so I don't need to reimplement Config UI on my system.
Related issues
https://github.com/apache/incubator-devlake/issues/2241
Are you willing to submit a PR?
- [X] Yes I am willing to submit a PR!
Code of Conduct
- [X] I agree to follow this project's Code of Conduct
@klesh @e2corporation
Hi, @mintsweet , I agree that we can try it out and see if it is viable, and evaluate the ROI.
Would you mind to share your Plan of Attack?
like creating a npm package for it, and using it in your project, step by step what do you plan to prove the concept?
For example:
- Refactor Jinkens connection editing page as an independent component
- Move it to a specific folder, create a
npmpackage for the folder - Reuse the package in other projects
- Summarize the effort we have to put in, and the value we may have in future development
In general, we need to know your plan so we can collaborate, and we need a report to justify the direction we are heading is worthwhile.
Hi, @klesh, this is my plan.
- Create a new directory
config-ui-lib.(This directory contains all the development bases, and I have done this step). - Create configuration related to Jenkins.
- Release-tested npm package of version 0.0.1-rc based on config-ui-lib.(This will be done by this week).
- Use the config-lib-ui package to integrate Jenkins into our system. (This will complete the trial by next week).
- If there is no problem with 4, I will make other connections like this. (It will take me a little more time).
- Finally I will use the new integrative config ui component to assemble the new page. (Then config-ui-lib will replace config-ui)
Hi, @klesh, this is my plan.
- Create a new directory
config-ui-lib.(This directory contains all the development bases, and I have done this step).- Create configuration related to Jenkins.
- Release-tested npm package of version 0.0.1-rc based on config-ui-lib.(This will be done by this week).
- Use the config-lib-ui package to integrate Jenkins into our system. (This will complete the trial by next week).
- If there is no problem with 4, I will make other connections like this. (It will take me a little more time).
- Finally I will use the new integrative config ui component to assemble the new page. (Then config-ui-lib will replace config-ui)
Sounds good. I would like to confirm a few things:
- Are all these changes happening in your PoC Branch?
- Will you prepare a report for ROI evaluation?
- Do we agree that the PoC branch will be merged if and only if we reached a consensus?
@klesh
- Yes.
- I need to think about it, maybe after finishing the first test I can give it.
- I don't know what counts as consensus.
@klesh
- Yes.
- I need to think about it, maybe after finishing the first test I can give it.
Yeah, sure.
- I don't know what counts as consensus.
Please read the definition https://dictionary.cambridge.org/dictionary/english/consensus
This issue has been automatically marked as stale because it has not had recent activity for 30 days. It will be closed in next 7 days if no further activity occurs.
This issue has been automatically marked as stale because it has not had recent activity for 30 days. It will be closed in next 7 days if no further activity occurs.
This issue has been automatically marked as stale because it has not had recent activity for 30 days. It will be closed in next 7 days if no further activity occurs.
This issue has been automatically marked as stale because it has not had recent activity for 30 days. It will be closed in next 7 days if no further activity occurs.
this job stopped.