Skip to content

Conversation

@bastianallgeier
Copy link
Member

@bastianallgeier bastianallgeier commented Jul 14, 2025

Description

This is part of the series of PRs to get rid of sections, but can be merged independently of the rest. It will move empty reports handling into the <k-stats> component to minimize the logic parts in the <k-stats-section>. It also removes some unused left-over code from <k-stats>

Changelog

Refactoring

  • Remove unused methods from <k-stats>
  • Move empty reports handling from <k-stats-section> to <k-stats>
  • Export props from <k-stats>

Breaking changes

None

Ready?

  • In-code documentation (wherever needed)
  • Unit tests for fixed bug/feature
  • Tests and CI checks all pass

For review team

  • Add lab and/or sandbox examples (wherever helpful)
  • Add changes & docs to release notes draft in Notion

@bastianallgeier bastianallgeier added this to the 5.1.0 milestone Jul 14, 2025
@bastianallgeier bastianallgeier requested a review from a team July 14, 2025 15:08
@distantnative distantnative merged commit 1f94b53 into develop-minor Jul 15, 2025
4 checks passed
@distantnative distantnative deleted the refact/empty-stats branch July 15, 2025 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants