Skip to content

refactor: migrate unit portfolios state to Angular component#448

Open
millyamolo wants to merge 2 commits intothoth-tech:10.0.xfrom
millyamolo:feat/angular17-portfolios-migration
Open

refactor: migrate unit portfolios state to Angular component#448
millyamolo wants to merge 2 commits intothoth-tech:10.0.xfrom
millyamolo:feat/angular17-portfolios-migration

Conversation

@millyamolo
Copy link
Copy Markdown

Replace the legacy AngularJS portfolios controller/template with a downgraded Angular component. This keeps the route while moving tab/filter/grading behavior onto the Angular 17 path.

Summary

  • Replace legacy AngularJS portfolios state controller/template logic with a downgraded Angular 17 component.
  • Add Angular component for tabs, filtering/search/sort/pagination, portfolio view, and grading workflow.
  • Keep route compatibility by using portfolios.tpl.html as a wrapper that renders <f-portfolios>.

Before / After Screenshots

Before (AngularJS)

  • Select Student tab
image - [ ] View Portfolio tab image - [ ] Assess Portfolio tab image

After (Angular 17)

  • Select Student tab
image - [ ] View Portfolio tab image - [ ] Assess Portfolio tab image ## Test Plan - [ ] Open `/units/:unitId/students/portfolios` - [ ] Select a student - [ ] Switch between tabs quickly (no hanging) - [ ] Validate filters/search/sort/pagination - [ ] Validate grade + Save flow

Replace the legacy AngularJS portfolios controller/template with a downgraded Angular component.
This keeps the route while moving tab/filter/grading behavior onto the Angular 17 path.

Made-with: Cursor
@millyamolo millyamolo changed the base branch from development to 10.0.x March 25, 2026 20:57
@millyamolo millyamolo closed this Mar 25, 2026
@millyamolo millyamolo reopened this Mar 25, 2026
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.

2 participants