Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New tech insights maturity plugin based on tech-insights #2464

Merged

Conversation

hballangan-mdsol
Copy link
Contributor

@hballangan-mdsol hballangan-mdsol commented Jan 8, 2025

Hey, I just made a Pull Request!

Create new a Tech Insights plugin extension called Tech Insights Maturity.
This is a frontend extension of tech-insights plugin and can be used alongside vanilla tech-insights-backend plugin or any of its implementations like tech-insights-backend-module-jsonfc.

image

(More screenshots also attached in readme)

For more info, see README

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)
  • All your commits have a Signed-off-by line in the message. (more info)

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
@backstage-goalie
Copy link
Contributor

backstage-goalie bot commented Jan 8, 2025

Missing Changesets

The following package(s) are changed by this PR but do not have a changeset:

  • @backstage-community/plugin-tech-insights-backend
  • @backstage-community/plugin-tech-insights-common
  • @backstage-community/plugin-tech-insights-maturity-common
  • @backstage-community/plugin-tech-insights-maturity
  • @backstage-community/plugin-tech-insights-node
  • @backstage-community/plugin-tech-insights

See CONTRIBUTING.md for more information about how to add changesets.

Changed Packages

Package Name Package Path Changeset Bump Current Version
@backstage-community/plugin-tech-insights-backend workspaces/tech-insights/plugins/tech-insights-backend none v2.1.1
@backstage-community/plugin-tech-insights-common workspaces/tech-insights/plugins/tech-insights-common none v0.4.0
@backstage-community/plugin-tech-insights-maturity-common workspaces/tech-insights/plugins/tech-insights-maturity-common none v0.1.0
@backstage-community/plugin-tech-insights-maturity workspaces/tech-insights/plugins/tech-insights-maturity none v0.1.0
@backstage-community/plugin-tech-insights-node workspaces/tech-insights/plugins/tech-insights-node none v2.2.0
@backstage-community/plugin-tech-insights workspaces/tech-insights/plugins/tech-insights none v0.4.0

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
@awanlin
Copy link
Contributor

awanlin commented Jan 10, 2025

Hi @hballangan-mdsol, would it make sense to just add this into the current tech-insights workspace instead of creating a new one considering the tight coupling?

@Xantier, what do you think?

@awanlin awanlin requested a review from Xantier January 10, 2025 21:06
@Xantier
Copy link
Contributor

Xantier commented Jan 13, 2025

Thanks for the contribution, looks stellar!

Yup, probably makes sense to use the existing workspace to keep everything more easily findable

@hballangan-mdsol
Copy link
Contributor Author

Hi @awanlin and @Xantier ! Thanks for checkin the PR! Makes sense to add it to tech-insights, is it okay to keep the naming for the 2 plugin folders when I move it there?

@Xantier
Copy link
Contributor

Xantier commented Jan 14, 2025

Is it okay to keep the naming for the 2 plugin folders when I move it there?

Yup, those look good to me

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
@hballangan-mdsol
Copy link
Contributor Author

Thanks @kurtaking for checking out unused components! I've removed all components not being used here, and renamed legacy files that were refactored.

@kurtaking
Copy link
Member

I think someone needs to approve running CI for the PR.

@hballangan-mdsol - once that happens, I think you are going to run into some failures based on the package.json and api reports.

I opened hballangan-mdsol#1 against your fork to show you the changes that were made when running

yarn backstage-cli repo fix --publish
yarn build:api-reports

hballangan-mdsol and others added 3 commits January 23, 2025 17:22
…i/MaturityApi.ts

Co-authored-by: Kurt King <kurtaking@gmail.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
@hballangan-mdsol
Copy link
Contributor Author

hballangan-mdsol commented Jan 24, 2025

I think someone needs to approve running CI for the PR.

@hballangan-mdsol - once that happens, I think you are going to run into some failures based on the package.json and api reports.

I opened hballangan-mdsol#1 against your fork to show you the changes that were made when running

yarn backstage-cli repo fix --publish
yarn build:api-reports

@kurtaking Thanks for the helpful fixes and suggestions! You've been very helpful in cleaning up this plugin. 🙏🏼

@Xantier aside from the configurable descriptions which we can update in the following version, will this this be good for merging?

@Xantier
Copy link
Contributor

Xantier commented Jan 24, 2025

@Xantier aside from the configurable descriptions which we can update in the following version, will this this be good for merging?

I'll take a last look at this today but from a cursory overview, this seems to look ok to ship as a first release

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This could be something to expose and inject to the client in the future if there are wants to modify the logic here

Xantier
Xantier previously approved these changes Jan 24, 2025
Copy link
Contributor

@Xantier Xantier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! :shipit:

@Xantier Xantier dismissed their stale review January 24, 2025 12:03

Conflicts in the PR

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
@kurtaking kurtaking mentioned this pull request Jan 24, 2025
5 tasks
@hballangan-mdsol
Copy link
Contributor Author

Nice! :shipit:

Resolved the conflicts and all checks are now passing. Kindly review again 🙏🏼 @Xantier

@Xantier Xantier merged commit 561d661 into backstage:main Jan 27, 2025
12 checks passed
@kurtaking
Copy link
Member

Nice work @hballangan-mdsol !

@hballangan-mdsol hballangan-mdsol deleted the feature/tech-insights-maturity branch January 28, 2025 04:55
@hballangan-mdsol
Copy link
Contributor Author

Nice work @hballangan-mdsol !

Thanks for all the help @kurtaking and @Xantier !
Looking forward to more collaborations in developing this plugin!

gaelgoth pushed a commit to gaelgoth/community-plugins that referenced this pull request Jan 31, 2025
)

* init tech-insights-maturity

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* add eslint

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* rename plugin export naming

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* update readme

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* MaturityApiRef to extend TechInsightsApiRef

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* sanitize and refactor

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* export EntityMaturitySummaryContent

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* fix remaining issue

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* update package dependencies

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* resolve use-react tsc issue

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* run prettier

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* generate build report

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* use workspace package

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* yarn fix

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* move tech-insights-maturity to  tech-insights workspace

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* refactor

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* add missing dependency

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* fix imported dependency

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* refactor getMaturityScore

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* remove custom org implementations

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* Apply suggestions from code review

Co-authored-by: Kurt King <kurtaking@gmail.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* refactor MaturityLink

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* remove non-common plugin dependencies in maturity-common

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* remove unused components

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* remove unused components

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* Update workspaces/tech-insights/plugins/tech-insights-maturity/src/api/MaturityApi.ts

Co-authored-by: Kurt King <kurtaking@gmail.com>
Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* update packages and report

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* update yarn packages

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* sync merge conflict

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* run yarn dedupe

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* generate api report md

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

* refactor maturityrank

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>

---------

Signed-off-by: Maica Ballangan <hballangan@mdsol.com>
Co-authored-by: Kurt King <kurtaking@gmail.com>
Signed-off-by: gaelgoth <gothuey.gael@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants