Skip to content

Refactor: Clean up unused props and dead code branches#341

Merged
RUKAYAT-CODER merged 4 commits intorinafcode:mainfrom
Nacho1499:fix/Unused-Component-Code
Apr 29, 2026
Merged

Refactor: Clean up unused props and dead code branches#341
RUKAYAT-CODER merged 4 commits intorinafcode:mainfrom
Nacho1499:fix/Unused-Component-Code

Conversation

@Nacho1499
Copy link
Copy Markdown
Contributor

closes #209

PR Description:
This PR addresses the cleanup of dead code branches and unused props within the Web3 component directory. It also modernizes the ESLint configuration to ensure long-term code quality.

Changes Made:
-Code Cleanup: Removed unreachable validation logic and unused props in .TransactionManager.tsx
-Hook Optimization: Refactored to resolve warnings and improved state synchronization.useWeb3Wallet.tsexhaustive-deps
-Config Migration: Moved ignore patterns from to to comply with ESLint 9+ standards..eslintignoreeslint.config.mjs
-Dependency Management: Integrated to resolve formatting conflicts.eslint-config-prettier

Acceptance Criteria Verified:
[x] No dead code branches remain.
[x] All component props are utilized.
[x] passes with zero warnings/errors.npm run lint

fix:
-Bug Fixes

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented Apr 28, 2026

@Nacho1499 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Nacho1499
Copy link
Copy Markdown
Contributor Author

lint

@RUKAYAT-CODER
Copy link
Copy Markdown
Contributor

Kindly resolve conflict

@Nacho1499 Nacho1499 force-pushed the fix/Unused-Component-Code branch from cc5611d to 5e06780 Compare April 28, 2026 15:53
@Nacho1499
Copy link
Copy Markdown
Contributor Author

Conflict can't be resolved. Because the conflict button is grayed out.

@Nacho1499
Copy link
Copy Markdown
Contributor Author

I have tried resolving it manually from my editor . But same result. But I will still try to resolving it. Thanks

@RUKAYAT-CODER
Copy link
Copy Markdown
Contributor

ok thanks

@Nacho1499 Nacho1499 force-pushed the fix/Unused-Component-Code branch from ee3e873 to 045fbed Compare April 29, 2026 08:09
@Nacho1499
Copy link
Copy Markdown
Contributor Author

conflict resolved

@RUKAYAT-CODER
Copy link
Copy Markdown
Contributor

Not yet resolved.
If the package-lock conflicts are messy,
resolve package.json first,
delete package-lock.json,
run npm install to regenerate it, test locally,
then pull/rebase before pushing.

@Nacho1499
Copy link
Copy Markdown
Contributor Author

conflict resolved . thanks

@RUKAYAT-CODER
Copy link
Copy Markdown
Contributor

Thank you

@RUKAYAT-CODER RUKAYAT-CODER merged commit b0a0d63 into rinafcode:main Apr 29, 2026
1 of 2 checks passed
@Nacho1499
Copy link
Copy Markdown
Contributor Author

nice working on ur repo. thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unused Component Code

2 participants