Closed Kodylow closed 3 weeks ago
apps/guardian-ui/src/components/setup/VerifyGuardians.tsx
[error] 121-122: Change to an optional chain. (lint/complexity/useOptionalChain) Unsafe fix: Change to an optional chain. --- [error] 236-347: This else clause can be omitted because previous branches break early. (lint/style/noUselessElse) Unsafe fix: Omit the else clause. --- [error] 238-347: This else clause can be omitted because previous branches break early. (lint/style/noUselessElse) --- [error] 249-347: This else clause can be omitted because previous branches break early. (lint/style/noUselessElse)apps/guardian-ui/src/components/dashboard/tabs/meta/ProposedMetas.tsx
[error] 181-187: This else clause can be omitted because previous branches break early. (lint/style/noUselessElse) Unsafe fix: Omit the else clause.apps/guardian-ui/src/components/setup/SetConfiguration.tsx
[error] 216-216: Avoid the use of spread (`...`) syntax on accumulators. (lint/performance/noAccumulatingSpread) Spread syntax should be avoided on accumulators (like those in `.reduce`) because it causes a time complexity of `O(n^2)`. Consider methods such as .splice or .push instead.
apps/guardian-ui/src/components/setup/ConnectGuardians.tsx (1)
`23-23`: **Updated SVG import path for CopyIcon** This change aligns with the PR's objective to fix broken asset paths by updating them to a more absolute path. This should ensure the icon is consistently loaded across different components.apps/guardian-ui/src/components/setup/VerifyGuardians.tsx (1)
`29-33`: **Updated SVG import paths for multiple icons** The changes to the import paths for `ArrowRightIcon`, `CopyIcon`, `CheckCircleIcon`, and `XCircleIcon` are consistent with the PR's goal to correct asset paths. This should help in maintaining a uniform reference structure across the project.apps/guardian-ui/src/components/dashboard/tabs/meta/ProposedMetas.tsx (1)
`19-19`: **Updated SVG import path for CheckIcon** This change updates the import path for `CheckIcon` to a more absolute path, which is in line with the PR's objective to fix asset paths. This ensures that the icon is loaded correctly across various parts of the application.apps/guardian-ui/src/components/setup/SetConfiguration.tsx (1)
`34-38`: **Updated SVG import paths for multiple icons** The changes to the import paths for `FedimintLogo`, `BitcoinLogo`, `ModulesIcon`, `ArrowRightIcon`, and `LightbulbLogo` correctly reflect the PR's goal to standardize asset paths across the project. This should help maintain consistency and ensure assets are loaded correctly.
fixes asset paths I broke in last refactor
Summary by CodeRabbit