fix: use the renamed CopyPaste icon export - #969
Merged
Conversation
A dependency bump of @sumup-oss/icons renamed the Copy export to CopyPaste, so CopyInput no longer type-checks: Module '"@sumup-oss/icons"' has no exported member 'Copy'. This fails astro check and has left the Lint workflow red on main, blocking every open pull request rather than only the one that bumped the dependency. Update the import and the render call to the new name. The icon is unchanged, so there is no visual difference.
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
developer-sumup-com | c56cca9 | Commit Preview URL Branch Preview URL |
Sep 09 2026, 10:51 AM |
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
1 participant
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.
Why
Lintis failing onmain(as ofc07471e), which blocks every open pull request, not just the one that introduced it:A Dependabot bump of
@sumup-oss/iconsrenamed theCopyexport toCopyPaste.CopyInput.tsxwas not updated with it, soastro checkfails.What changed
Two lines in
src/components/Forms/CopyInput/CopyInput.tsx— the import and therenderSuffixcall — now useCopyPaste. ConfirmedCopyPasteis the only copy-related export in the installed version. The icon itself is unchanged, so there is no visual difference.Verification
Full matrix for a source change:
npm run check— 0 errors, 0 warnings (was 1 error).npm run lint— clean, bothlint:srcandlint:markdown.npm run test— 10 tests passed across 2 files.Note
Found while working on #968. Kept separate so that documentation change stays content-only, and so this can merge on its own to unblock other PRs.