Fix/manage solution file picker requestid#43
Conversation
…iguration and update copyright notices
…solution-webview-main files
…d update file selection data format
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
|
Coverage Impact ⬆️ Merging this pull request will increase total coverage on Modified Files with Diff Coverage (2)
🤖 Increase coverage with AI coding...🚦 See full report on Qlty Cloud » 🛟 Help
|
There was a problem hiding this comment.
Pull request overview
This PR aims to fix/modernize Manage Solution’s file-picker message correlation (per PR description) and includes a small set of supporting type/test updates in the extension/webview code.
Changes:
- Reordered Manage Solution component CSS imports.
- Updated a Manage Solution webview main unit test to pass an explicit
configurationProvider. - Added a global TypeScript declaration for importing
*.cssfiles and updated a solution manager test factory to include theactivatedload-state flag.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/views/manage-solution/view/components/manage-solution.tsx | Import ordering change for CSS files in the Manage Solution React component. |
| src/views/manage-solution/manage-solution-webview-main.test.ts | Adjusts test wiring to pass configurationProvider into the factory. |
| src/style.d.ts | Adds declare module '*.css' to support CSS imports in TS. |
| src/solutions/solution-manager.factories.ts | Extends SolutionLoadState factories to include the new activated field. |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 7 out of 8 changed files in this pull request and generated 2 comments.
You can also share your feedback on Copilot code review. Take the survey.
…d corresponding test

Refactor Manage Solution file picker messaging to use [requestId] instead of DOM element IDs.
Fixes
Changes
Screenshots
Checklist