Skip to content

fix: removed console errors#2437

Merged
rsodre merged 3 commits intomainfrom
rsodre/fix-console-errors
Feb 23, 2026
Merged

fix: removed console errors#2437
rsodre merged 3 commits intomainfrom
rsodre/fix-console-errors

Conversation

@rsodre
Copy link
Collaborator

@rsodre rsodre commented Feb 23, 2026

  • removed google fonts css import error
  • log ekubo quote errors as warnings
  • stop fetching ekubo quotes for invalidated tokens (non-existing, no liquidity)

@vercel
Copy link

vercel bot commented Feb 23, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
controller-example-next Ready Ready Preview Feb 23, 2026 4:59pm
keychain Ready Ready Preview Feb 23, 2026 4:59pm
keychain-storybook Ready Ready Preview Feb 23, 2026 4:59pm

Request Review

@rsodre rsodre marked this pull request as ready for review February 23, 2026 17:08
@rsodre rsodre enabled auto-merge (squash) February 23, 2026 17:08
@rsodre rsodre merged commit 4374082 into main Feb 23, 2026
13 checks passed
@rsodre rsodre deleted the rsodre/fix-console-errors branch February 23, 2026 22:04
@claude
Copy link
Contributor

claude bot commented Feb 23, 2026

Claude finished @rsodre's task —— View job


Documentation Analysis

Checklist:

  • Analyze PR changes
  • Determine if user-facing behavior changed
  • Update documentation if needed

Changes in PR #2437:

  • Removed Google fonts CSS import error (internal HTML)
  • Changed Ekubo quote errors to warnings (internal logging)
  • Added token invalidation logic to prevent repeated failed API calls (internal optimization)

Assessment:
No documentation updates needed. All changes are internal implementation details:

  • HTML/CSS fixes don't affect user-facing APIs
  • Logging changes (error → warning) are not visible to end users
  • Token invalidation is an internal performance optimization with no API or behavioral changes

Per documentation rule #1 (DEFAULT TO NO CHANGES): Internal refactors, bug fixes, and performance improvements do not require documentation updates.


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.

2 participants