achievements: add cv attachment (fixes #9782)#9789
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 100a7db52a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
Pull request overview
Adds support for attaching and viewing a CV/Resume PDF on the “My Achievements” feature, addressing #9782 by surfacing a “View CV/Resumé” action and providing upload/remove controls in the achievements update flow.
Changes:
- Add a “View CV/Resumé” button on the achievements page when a resume attachment exists.
- Add resume upload UI + client-side PDF validation and submission logic to store the attachment.
- Extend the shared
planet-file-inputcomponent with anacceptinput and updated layout styling.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/app/users/users-achievements/users-achievements.scss | Adjust toolbar/button layout styles for responsiveness. |
| src/app/users/users-achievements/users-achievements.component.ts | Add computed resumeUrl for linking to a CouchDB attachment. |
| src/app/users/users-achievements/users-achievements.component.html | Add “View CV/Resumé” toolbar action when a resume is present. |
| src/app/users/users-achievements/users-achievements-update.scss | Add layout styling for existing/new file controls. |
| src/app/users/users-achievements/users-achievements-update.component.ts | Add resume selection/validation, unsaved-change tracking, and attachment posting behavior. |
| src/app/users/users-achievements/users-achievements-update.component.html | Add resume upload/removal controls to the update screen. |
| src/app/shared/forms/file-input.component.ts | Add accept input and adjust component template/styles. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 7 out of 7 changed files in this pull request and generated 6 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
src/app/users/users-achievements/users-achievements-update.component.html
Show resolved
Hide resolved
src/app/users/users-achievements/users-achievements-update.component.ts
Outdated
Show resolved
Hide resolved
src/app/users/users-achievements/users-achievements-update.component.ts
Outdated
Show resolved
Hide resolved
src/app/users/users-achievements/users-achievements.component.html
Outdated
Show resolved
Hide resolved
src/app/users/users-achievements/users-achievements-update.component.html
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Fixes #9782
Add CV/Resume attachments to myAchievements.