Skip to content

Implement Pie widget#384

Merged
mum4k merged 8 commits intomum4k:develfrom
VincenzoManto:master
Sep 28, 2025
Merged

Implement Pie widget#384
mum4k merged 8 commits intomum4k:develfrom
VincenzoManto:master

Conversation

@VincenzoManto
Copy link

As in #383 this is an implementation of a Pie widget (better a donut multi values)

Copy link
Owner

@mum4k mum4k left a comment

Choose a reason for hiding this comment

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

Thank you for the contribution!

@mum4k mum4k changed the base branch from master to devel August 26, 2025 00:23
@VincenzoManto
Copy link
Author

@mum4k thank you so much for your review. I pushed new changes. Please, tell me if there is something else to correct, I'll fix it

Copy link
Owner

@mum4k mum4k left a comment

Choose a reason for hiding this comment

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

Thank you for making the changes, looks good. We have two remaining comments, both optional) if you want to take a look before we merge this in.

@VincenzoManto
Copy link
Author

Hi @mum4k, thanks again for your review. If you need anything else, just let me know!

@mum4k
Copy link
Owner

mum4k commented Sep 28, 2025

Once again, thank you for the contribution.

@mum4k mum4k merged commit bc63544 into mum4k:devel Sep 28, 2025
4 checks passed
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.

2 participants