Skip to content

refactor(dashboard): migrate student dashboard components to Angular/TypeScript#1177

Open
m1chael-pappas wants to merge 1 commit intodoubtfire-lms:10.0.xfrom
m1chael-pappas:migrate/student-dashboard
Open

refactor(dashboard): migrate student dashboard components to Angular/TypeScript#1177
m1chael-pappas wants to merge 1 commit intodoubtfire-lms:10.0.xfrom
m1chael-pappas:migrate/student-dashboard

Conversation

@m1chael-pappas
Copy link
Copy Markdown

Description

Migrate the student dashboard components from CoffeeScript/AngularJS to Angular/TypeScript. This includes student-task-list, progress-dashboard, and their chart dependencies (progress-burndown-chart, student-task-status-pie-chart). Also creates a reusable NVD3 wrapper component and VisualisationService that can be used for future chart migrations. Removes dead code (project-tasks-list, project-progress-dashboard). Fixes burndownChartData type in Project model.

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Logged in as student_0 and navigated to Introduction to Programming project dashboard. Verified task list renders with search filtering, status badges, and task selection. Verified progress dashboard shows target grade selector, burndown chart, and task status pie chart. Verified clicking tasks opens the task dashboard panel.

Testing Checklist:

  • Tested in latest Chrome
  • Tested in latest Safari
  • Tested in latest Firefox

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 in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have requested a review from @macite and @jakerenzella on the Pull Request

…TypeScript

Migrate student-task-list, progress-dashboard, progress-burndown-chart,
and student-task-status-pie-chart from CoffeeScript to Angular. Create
reusable NVD3 wrapper component and VisualisationService for future
chart migrations. Remove unused project-tasks-list and
project-progress-dashboard. Fix burndownChartData type in Project model.
@m1chael-pappas
Copy link
Copy Markdown
Author

Ready for review. @macite @jakerenzella

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