Skip to content

🩹 app: fix legacy read contract query error#921

Merged
cruzdanilo merged 1 commit intomainfrom
query
Mar 27, 2026
Merged

🩹 app: fix legacy read contract query error#921
cruzdanilo merged 1 commit intomainfrom
query

Conversation

@dieguezguille
Copy link
Copy Markdown
Member

@dieguezguille dieguezguille commented Mar 27, 2026


Open with Devin

Summary by CodeRabbit

  • Bug Fixes

    • Fixed a legacy "read contract" query error that could surface in the app.
  • Chores

    • Updated default query behavior to prevent unnecessary automatic refetches and reduce unexpected refreshes.

@dieguezguille dieguezguille requested a review from franm91 as a code owner March 27, 2026 19:01
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 27, 2026

🦋 Changeset detected

Latest commit: d4ec64d

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 27, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 04fb2cae-e90c-4333-856b-c5e0e72c4593

📥 Commits

Reviewing files that changed from the base of the PR and between 97f14bd and d4ec64d.

📒 Files selected for processing (2)
  • .changeset/sour-words-happen.md
  • src/utils/queryClient.ts

Walkthrough

Added a React Query default for the ["readContract"] query key that provides a queryFn which throws Error("don't refetch"), preventing automatic refetching and matching existing suppression behavior.

Changes

Cohort / File(s) Summary
Query Client Configuration
src/utils/queryClient.ts
Added a default React Query configuration for the ["readContract"] key with a queryFn that immediately throws Error("don't refetch") to suppress automatic refetching.
Changeset
.changeset/sour-words-happen.md
Added a changeset bumping @exactly/mobile (patch) noting a fix for a legacy read contract query error.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Suggested reviewers

  • franm91
🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: fixing a legacy read contract query error by adding a configuration to prevent automatic refetching.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch query

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

devin-ai-integration[bot]

This comment was marked as resolved.

gemini-code-assist[bot]

This comment was marked as resolved.

@sentry
Copy link
Copy Markdown

sentry bot commented Mar 27, 2026

✅ All tests passed.

@dieguezguille dieguezguille changed the title 🐛 app: fix orphaned read contract query error 🩹 app: fix legacy read contract query error Mar 27, 2026
coderabbitai[bot]

This comment was marked as resolved.

@cruzdanilo cruzdanilo merged commit d4ec64d into main Mar 27, 2026
3 of 6 checks passed
@cruzdanilo cruzdanilo deleted the query branch March 27, 2026 19:50
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