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

fix(assets): split asset utilization plugin code from asset calibration plugin code #65

Merged
merged 4 commits into from
Sep 9, 2024

Conversation

kkerezsi
Copy link
Collaborator

@kkerezsi kkerezsi commented Sep 8, 2024

Pull Request

🤨 Rationale

The following PR splits the current asset Grafana plugin into 2 separate plugins:

  • Asset Utilization plugin
  • Asset Calibration plugin

Link to workitem: Bug 2847424: Add feature flag to hide current progress on the Grafana Assets plugin

👩‍💻 Implementation

  • New plugin was generated
  • Code related to Calibration features are migrated from existing shared plugin to the Asset Calibration Plugin

🧪 Testing

  • Unit test
  • Manual tests

✅ Checklist

@kkerezsi kkerezsi changed the title Split Asset Utilization Plugin From Asset Calibration Plugin fix(assets): split asset utilization plugin code from asset calibration plugin code Sep 8, 2024
@mure mure merged commit 2c99c56 into main Sep 9, 2024
1 check passed
mure pushed a commit that referenced this pull request Sep 9, 2024
## [3.12.1](v3.12.0...v3.12.1) (2024-09-09)

### Bug Fixes

* **assets:** split asset utilization plugin code from asset calibration plugin code ([#65](#65)) ([2c99c56](2c99c56))
@mure
Copy link
Collaborator

mure commented Sep 9, 2024

🎉 This PR is included in version 3.12.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@mure mure added the released label Sep 9, 2024
@mure mure mentioned this pull request Sep 9, 2024
1 task
@mure mure deleted the users/alkerezs/implement-feature-flags-asset-plugin branch October 3, 2024 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants