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.

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

chore: Remove warnings and deprecated info graphql components #11124

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

memoyil
Copy link
Collaborator

@memoyil memoyil commented Jan 9, 2025


PR-Codex overview

This PR primarily focuses on code cleanup and refactoring. It removes unused files and simplifies existing code, improving readability and maintainability while also updating certain functions and hooks to enhance performance.

Detailed summary

  • Deleted unused files: getEthPrices.ts, tokenData.ts, topTokens.ts.
  • Refactored handleConnectWallet functions to use _ instead of e.
  • Simplified dependencies in various components.
  • Removed unused variables and parameters in hooks and functions.
  • Updated graph query logic to eliminate redundant code.
  • Adjusted conditions for stable swap checks in multiple locations.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Copy link

changeset-bot bot commented Jan 9, 2025

⚠️ No Changeset found

Latest commit: 0092f20

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

Copy link

vercel bot commented Jan 9, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
web 🛑 Canceled (Inspect) 💬 Add feedback Feb 13, 2025 11:21am
7 Skipped Deployments
Name Status Preview Comments Updated (UTC)
aptos-web ⬜️ Ignored (Inspect) Visit Preview Feb 13, 2025 11:21am
blog ⬜️ Ignored (Inspect) Visit Preview Feb 13, 2025 11:21am
bridge ⬜️ Ignored (Inspect) Visit Preview Feb 13, 2025 11:21am
games ⬜️ Ignored (Inspect) Visit Preview Feb 13, 2025 11:21am
gamification ⬜️ Ignored (Inspect) Visit Preview Feb 13, 2025 11:21am
uikit ⬜️ Ignored (Inspect) Visit Preview Feb 13, 2025 11:21am
ton ⬜️ Skipped (Inspect) Feb 13, 2025 11:21am

@memoyil memoyil force-pushed the feature/chore_remove_warnings_and_unused_info_gql branch from 11622e0 to a1a3bee Compare January 9, 2025 07:03
@vercel vercel bot temporarily deployed to Preview – ton January 9, 2025 07:03 Inactive
@memoyil memoyil force-pushed the feature/chore_remove_warnings_and_unused_info_gql branch from a1a3bee to 15871ac Compare January 9, 2025 07:04
@vercel vercel bot temporarily deployed to Preview – ton January 9, 2025 07:04 Inactive
@vercel vercel bot temporarily deployed to Preview – ton January 9, 2025 07:18 Inactive
@memoyil memoyil force-pushed the feature/chore_remove_warnings_and_unused_info_gql branch from 4123db6 to a52a258 Compare February 13, 2025 10:53
@vercel vercel bot temporarily deployed to Preview – ton February 13, 2025 10:54 Inactive
@vercel vercel bot temporarily deployed to Preview – ton February 13, 2025 11:03 Inactive
@memoyil memoyil force-pushed the feature/chore_remove_warnings_and_unused_info_gql branch from b6011fd to 00d0176 Compare February 13, 2025 11:05
@vercel vercel bot temporarily deployed to Preview – ton February 13, 2025 11:05 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant