Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Core: improve manager-entries failure message #20306

Merged
merged 2 commits into from
Dec 16, 2022
Merged

Core: improve manager-entries failure message #20306

merged 2 commits into from
Dec 16, 2022

Conversation

yannbf
Copy link
Member

@yannbf yannbf commented Dec 16, 2022

Issue: N/A

What I did

This PR improves the error message when manager entries (e.g. addons that use manager) are broken

How to test

  • Is this testable with Jest or Chromatic screenshots?
  • Does this need a new example in the kitchen sink apps?
  • Does this need an update to the documentation?

If your answer is yes to any of these, please make sure to include it in your PR.

@yannbf yannbf added maintenance User-facing maintenance tasks core labels Dec 16, 2022
@yannbf yannbf merged commit b2cb33d into next Dec 16, 2022
@yannbf yannbf deleted the fix/manager-entries-failure-message branch December 16, 2022 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ndelangen ndelangen ndelangen approved these changes

Assignees

@ndelangen ndelangen

Labels
core maintenance User-facing maintenance tasks
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@yannbf @ndelangen