Skip to content

[PER-10415][BLOCKED BY BE] Support multiple Archive Stewards per archive#1048

Open
aasandei-vsp wants to merge 1 commit into
mainfrom
PER-10432-multiple-stewards
Open

[PER-10415][BLOCKED BY BE] Support multiple Archive Stewards per archive#1048
aasandei-vsp wants to merge 1 commit into
mainfrom
PER-10432-multiple-stewards

Conversation

@aasandei-vsp
Copy link
Copy Markdown
Contributor

Adapt the Legacy Planning panel to render one card per Archive Steward instead of a single steward record. Each card has a $PR-blue-100 border, a $PR-blue-25 hover background, and a material-icons edit icon in the top-right; the whole card is clickable and reuses the existing edit form to update that steward. The bottom button becomes "Add a new Archive Steward" and opens the same form with no pre-filled directive. The legacy-contact requirement now only gates the very first steward; once at least one exists, additional ones can be added without it. Errors when loading or saving stewards surface through MessageService.showError.

Issue: PER-10415

Adapt the Legacy Planning panel to render one card per Archive Steward
instead of a single steward record. Each card has a $PR-blue-100 border,
a $PR-blue-25 hover background, and a material-icons edit icon in the
top-right; the whole card is clickable and reuses the existing edit form
to update that steward. The bottom button becomes "Add a new Archive
Steward" and opens the same form with no pre-filled directive. The
legacy-contact requirement now only gates the very first steward; once
at least one exists, additional ones can be added without it. Errors
when loading or saving stewards surface through MessageService.showError.

Issue: PER-10415
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 4, 2026

Codecov Report

❌ Patch coverage is 95.12195% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 50.13%. Comparing base (e929778) to head (777e101).
⚠️ Report is 6 commits behind head on main.

Files with missing lines Patch % Lines
...nts/directive-dialog/directive-dialog.component.ts 90.00% 1 Missing ⚠️
...s/directive-display/directive-display.component.ts 94.44% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1048      +/-   ##
==========================================
+ Coverage   50.01%   50.13%   +0.12%     
==========================================
  Files         348      348              
  Lines       11502    11528      +26     
  Branches     1975     1984       +9     
==========================================
+ Hits         5753     5780      +27     
+ Misses       5564     5560       -4     
- Partials      185      188       +3     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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