Improve UI/copy for per-package error messages#192
Merged
Conversation
The *.tsx files in src/ui were not being properly linted. Now they are, and lint violations have been corrected. Additionally, `vite.config.mjs` was renamed to `vite.config.mts` so that it could be properly linted as well.
When the user chooses to include a package in the release, they may be asked to also include dependencies or dependents of that package. The UI for these error messages, as well as the messages themselves, is not as user-friendly as it could be. This commit adds an expandable "help text" for each kind of error to explain what actions the user should take and why.
mcmire
force-pushed
the
improve-per-package-error-messages
branch
from
3726a8b to
6482092
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
When the user chooses to include a package in the release, they may be asked to also include dependencies or dependents of that package. The UI for these error messages, as well as the messages themselves, is not as user-friendly as it could be. This commit adds an expandable "help text" for each kind of error to explain what actions the user should take and why.
Screenshots
Before
After
Note
Improves clarity and usability of per-package dependency errors with structured summaries and expandable guidance.
descriptionwitherrorSubjectanderrorDetailsinDependencyErrorSectionand updates layout/styles (red header, top border, "Read more" expandable details)package-<name>and "Skip"/"Skip All" actionsPackageItem, uses the newDependencyErrorSectionfor bothMissing DependenciesandMissing Peer Dependentswith expanded, actionable copyWritten by Cursor Bugbot for commit abb289f. This will update automatically on new commits. Configure here.