figma-plugin icon indicating copy to clipboard operation
figma-plugin copied to clipboard

When we pull tokens and we show the pull dialog and we hit an error, show an error screen

Open Copilot opened this issue 9 months ago • 3 comments

Thanks for assigning this issue to me. I'm starting to work on it and will keep this PR's description up to date as I form a plan and make progress.

Original issue description:

In the PullDialog when we have an error pulling, let's show the error callout we also show on the start screen that lets them understand what was going on and what's the issue.

Fixes #3383.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot avatar May 28 '25 08:05 Copilot

⚠️ No Changeset found

Latest commit: 8b620f930f6dc6e5515be021b3f4c0d23d71d4ef

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

changeset-bot[bot] avatar May 28 '25 08:05 changeset-bot[bot]

try again

six7 avatar May 28 '25 10:05 six7

try again

I've implemented the error handling for the PullDialog as requested. When a pull operation fails, the dialog now shows an error callout similar to the one on the start screen, providing clear feedback about what went wrong. The implementation handles various error scenarios including provider failures, network issues, and empty responses. Changes include error state support, comprehensive test coverage, and proper translations. Commit: 8b620f9

Copilot avatar May 28 '25 11:05 Copilot