Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #958 +/- ##
==========================================
+ Coverage 48.69% 48.87% +0.17%
==========================================
Files 350 352 +2
Lines 11339 11370 +31
Branches 1898 1905 +7
==========================================
+ Hits 5522 5557 +35
+ Misses 5627 5620 -7
- Partials 190 193 +3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
e8816ac to
0b70012
Compare
Add a button on the profile editor to sort milestones chronologically or reverse chronologically when they appear on the public profile.
755b2ff to
0f764af
Compare
|
Loom for @omnignorant or anyone else: https://www.loom.com/share/c8d1f8f1cf6643c4a49f02e164263750 I should have turned off audio, but you can mute it - there's no narration. Or you can hear me clicking around if you put the sound on. :) |
There was a problem hiding this comment.
Pull request overview
Adds support for choosing whether milestones on a public profile are displayed chronologically (oldest-first) or reverse-chronologically (newest-first), with a toggle exposed in the profile edit UI and persisted to the archive via a V2 PATCH.
Changes:
- Add
milestoneSortOrdertoArchiveVOand use it to drive milestone sorting on the public profile. - Add a profile-edit toggle button + saving state that PATCHes the archive setting.
- Add unit tests covering both the public sorting behavior and the toggle behavior.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
src/app/shared/services/api/archive.repo.ts |
Adds patchArchive() to persist milestoneSortOrder via PATCH v2/archive/:id. |
src/app/public/components/public-profile/public-profile.component.ts |
Sorts milestones asc/desc based on archive.milestoneSortOrder. |
src/app/public/components/public-profile/public-profile.component.spec.ts |
New tests verifying default/newest-first vs chronological/oldest-first sorting. |
src/app/models/archive-vo.ts |
Adds milestoneSortOrder field to the archive model. |
src/app/core/components/profile-edit/profile-edit.component.ts |
Implements toggleMilestoneOrder() and saving guard flag. |
src/app/core/components/profile-edit/profile-edit.component.spec.ts |
Adds unit tests for toggle behavior and saving flag. |
src/app/core/components/profile-edit/profile-edit.component.html |
Adds reorder button to Milestones section and updates help text. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
aasandei-vsp
left a comment
There was a problem hiding this comment.
Just a small comment.
| const newOrder = | ||
| this.archive.milestoneSortOrder === 'chronological' | ||
| ? 'reverse_chronological' | ||
| : 'chronological'; |
There was a problem hiding this comment.
chronological and reverse_chronological seem to repeat in multiple places. I would create an enum or at least a type for them.
|
Holding off on this for some design help from @omnignorant. |
|
Existing text in the profile editor under the Milestones heading:
Design guidance for milestone sort toggle.
|
Allow users to display milestones on the public profile either chronologically or reverse chronologically.
I could use some help with making it obvious to the user (1) that this is a button and (2) what the button does. I don't know how to make sure that they understand that they are reordering the milestones on a different screen than the one they are looking at.