Skip to content

Add download/upload buttons to MSEL info page and upload warning - #250

Merged
sei-tspencer merged 5 commits into
mainfrom
feature/msel-info-export-import
May 12, 2026
Merged

Add download/upload buttons to MSEL info page and upload warning#250
sei-tspencer merged 5 commits into
mainfrom
feature/msel-info-export-import

Conversation

@sei-awelle

Copy link
Copy Markdown
Contributor

Add download menu (JSON/XLSX) and XLSX upload button to the MSEL info Config tab. Show confirmation warning when uploading XLSX to an existing MSEL since it overwrites data fields and scenario events.

Add download menu (JSON/XLSX) and XLSX upload button to the MSEL
info Config tab. Show confirmation warning when uploading XLSX to
an existing MSEL since it overwrites data fields and scenario events.
@sei-awelle
sei-awelle requested a review from a team as a code owner April 1, 2026 19:25
Comment thread src/app/components/msel-info/msel-info.component.html
Add JSON and XLSX download options to msel-info Config tab.
Remove destructive XLSX upload that would overwrite existing MSELs:
- Removed upload button from msel-info page
- Removed per-row upload button from msel-list page
- Kept safe 'create new MSEL' upload functionality

Users can now export MSELs for review but cannot accidentally
overwrite data fields, scenario events, or competency mappings.
Add getDeleteTooltip() method to both msel-list and msel-info components.
Tooltip explains when delete is disabled:
- Deployed MSELs cannot be deleted
- Template MSELs cannot be deleted
- User lacks permission

Also add isTemplate check to msel-info delete button disabled condition
to match msel-list behavior.
Seed data MSELs have createdBy = 00000000-0000-0000-0000-000000000000.
Skip API call for empty GUID and set creator name to 'unknown'.
@sei-tspencer
sei-tspencer merged commit e90c6bb into main May 12, 2026
7 checks passed
@sei-tspencer
sei-tspencer deleted the feature/msel-info-export-import branch May 12, 2026 11:58
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