Skip to content

Replace eChartLabel enum with CHART_LABELS and ChartMetric - #2025

Open
mutatrum wants to merge 2 commits into
bitaxeorg:masterfrom
mutatrum:refactor/axe-os-chart-labels
Open

mutatrum wants to merge 2 commits into
bitaxeorg:masterfrom
mutatrum:refactor/axe-os-chart-labels

Conversation

@mutatrum

@mutatrum mutatrum commented Oct 8, 2026

Copy link
Copy Markdown
Collaborator

Uses const disctionary instead of display labels for lookups.

Replaces eChartLabel enum with CHART_LABELS and ChartMetric in chart-labels.ts
Removes chartLabelKey and chartLabelValue conversion helpers
Updates HomeComponent and SystemApiService to use canonical metrics directly

…tMetric

- Replace eChartLabel enum with CHART_LABELS and ChartMetric in src/models/chart-labels.ts
- Remove chartLabelKey and chartLabelValue conversion helpers
- Update HomeComponent and SystemApiService to use canonical metrics directly
@github-actions

github-actions Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Test Results

  2 files  ±0    2 suites  ±0   2s ⏱️ -1s
256 tests ±0  256 ✅ ±0  0 💤 ±0  0 ❌ ±0 
258 runs  ±0  258 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 5b84ac6. ± Comparison against base commit 7daa022.

♻️ This comment has been updated with latest results.

# Conflicts:
#	main/http_server/axe-os/src/app/components/home/home.component.ts
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.

1 participant