Skip to content

fix: null guard for r.status and fix aria-valuemax in admin-analytics.js - #7548

Merged
janavipandole merged 1 commit into
janavipandole:mainfrom
tmdeveloper007:fix/#7513-7517-admin-analytics-clean
Aug 13, 2026
Merged

fix: null guard for r.status and fix aria-valuemax in admin-analytics.js#7548
janavipandole merged 1 commit into
janavipandole:mainfrom
tmdeveloper007:fix/#7513-7517-admin-analytics-clean

Conversation

@tmdeveloper007

Copy link
Copy Markdown
Contributor

📄 Description

Fixed the missing closing double-quote on aria-valuemax in renderStatusDistribution. Also added a null/undefined guard for r.status that returns an empty string, preventing undefined labels in the analytics dashboard.

🔗 Related Issues

Closes #7513
Closes #7517

🧩 Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Style / UI improvement
  • Refactor
  • Test
  • Chore / dependency update

✅ Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Note: This task is being handled by tmdeveloper007 as part of GSSOC (GirlScript Summer of Code) — please assign this PR to the tmdeveloper007 account when picking it up.

@tmdeveloper007 tmdeveloper007 changed the title fix : null guard for r.status and fix aria-valuemax in admin-analytics.js fix: null guard for r.status and fix aria-valuemax in admin-analytics.js Aug 12, 2026
@janavipandole
janavipandole merged commit 52d02e0 into janavipandole:main Aug 13, 2026
21 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

2 participants