Skip to content

Address site Accessibility Issues#321

Open
ronardcaktus wants to merge 7 commits intodevelopfrom
320-accessibility-issues
Open

Address site Accessibility Issues#321
ronardcaktus wants to merge 7 commits intodevelopfrom
320-accessibility-issues

Conversation

@ronardcaktus
Copy link
Member

@ronardcaktus ronardcaktus commented Mar 3, 2026

This PR addresses several accessibility issues flagged by an external tool and reported to the client. The sites that have accessibility issues are listed in the issue, along with a PDF the client shared with us.

Note: While fixing these I noticed that the url health-alerts/ was pretty slow so I went ahead and optimized it.

Closes #320

@ronardcaktus
Copy link
Member Author

ronardcaktus commented Mar 9, 2026

Before Health Alert Query optimization

from django.test import RequestFactory
from apps.health_alerts.models import HealthAlertListPage

request = RequestFactory().get('/')
halp = HealthAlertListPage.objects.first()
halp.get_context(request)

context = halp.get_context(request)
context['health_alerts'].count()
443

After Health Alert Query optimization

context['health_alerts'].count()
Out[7]: 443

The same number of queries is being returned, showing that the optimization didn't seem to break anything.

@ronardcaktus ronardcaktus requested a review from HusnaHariz March 9, 2026 19:59
@ronardcaktus ronardcaktus marked this pull request as ready for review March 10, 2026 13:05
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.

Address site Accessibility Issues

1 participant