Repository navigation
Move RegionalAnalyticsView out of RoleDashboards.tsx #139
Description
Activity
@jgupta05072003-code
Respected Ma'am,
I would like to work on this issue, can you please confirm so I can start working on this towards phase 2 of my internship. I had started my internship on 01 June, 2026, and if a PR merge is compulsory then please let me work on this get the PR merged, then I will still keep on working after completing phase 2, on a separate new feature for defaulted schools that I am developing. I am requesting this due to urgent requirement of internship completion certificate. It would be so kind if you would kindly review the PR on this after affirming I can work on this issue #136
I already have completed all other requirements for previous steps.
I would forever be grateful for your kind response in this regards at your earliest convenience.Thanking You,
Nitish Naveen
Indian Institute of Information Technology (IIIT) DharwadSaw PR #230 is already open for this — leaving it with you, not touching it myself.
Could you get it merge-ready by end of day tomorrow (2026-08-20)? If you hit a blocker, just comment here rather than letting it sit.
Yes ma'am, I'm working and will update from my side shortly. The person raising the PR #230 has not submitted the onboarding.md as well as I think they might be a French open-source contributor, though I had requested and was approved from you yesterday to work on this issue, I'll do the same and raise a PR shortly.
Thank You@jgupta05072003-code Ma'am please review PR #241
- Onboarding.md document merged
- Approved from your side
- Screenshots in PR
- All checks passed, no merge conflicts
- No logic change so easy merge without much efforts
Nitish Naveen
Intern, Vicharanashala
Student, IIIT Dharwad
What this is for
RoleDashboards.tsxis a 3,100+ line file holding all 8 role dashboards. We're moving each piece into its own file so the codebase is easier to navigate.What to move
export const RegionalAnalyticsView(~292 lines) — a shared analytics view used by more than one dashboard.Steps
frontend/src/components/dashboards/RegionalAnalyticsView.tsx.RoleDashboards.tsxinto the new file, with whatever imports it needs.RoleDashboards.tsx, add:export { RegionalAnalyticsView } from './dashboards/RegionalAnalyticsView';where it used to be.Watch out for
RegionalAnalyticsView(search the file for it) before you start, so you know which screens to test afterward.How to check your work
cd frontend && npm run lint— no errors.npm run build— succeeds.npm run devand check every dashboard that uses this view — confirm they all still render it correctly.Part of #136.