Skip to content

feat: toggle SUI flag#11519

Merged
gomesalexandre merged 2 commits intodevelopfrom
claude/toggle-sui-flag-XGT9M
Dec 26, 2025
Merged

feat: toggle SUI flag#11519
gomesalexandre merged 2 commits intodevelopfrom
claude/toggle-sui-flag-XGT9M

Conversation

@gomesalexandre
Copy link
Copy Markdown
Contributor

@gomesalexandre gomesalexandre commented Dec 23, 2025

Description

Does the thing

Issue (if applicable)

closes #

Risk

High Risk PRs Require 2 approvals

What protocols, transaction types, wallets or contract interactions might be affected by this PR?

Testing

  • smoke test SUI

Engineering

Operations

  • 🏁 My feature is behind a flag and doesn't require operations testing (yet)

Screenshots (if applicable)

https://jam.dev/c/29f80dd3-a44c-417c-954f-914a5b7e50ac

Summary by CodeRabbit

  • Chores
    • Enabled SUI feature in production environment and disabled it in development configuration.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Dec 23, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The SUI feature flag was enabled in .env (set to true) and the explicit VITE_FEATURE_SUI=true line was removed from .env.development, leaving development to inherit the production/default setting.

Changes

Cohort / File(s) Summary
Environment Configuration
.env, .env.development
.env: VITE_FEATURE_SUI changed from falsetrue. .env.development: removed VITE_FEATURE_SUI=true override.

Sequence Diagram(s)

(omitted — change is a configuration toggle without multi-component control-flow)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • NeOMakinG
  • 0xApotheosis

Poem

🐰✨ I flipped a switch, the config sings,
SUI hops in on nimble springs.
Dev no longer shouts its line,
Prod's bright flag will now align—
Hooray, code carrots and new things! 🥕

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch claude/toggle-sui-flag-XGT9M

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 5528717 and 62ed76f.

📒 Files selected for processing (2)
  • .env
  • .env.development

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.

@gomesalexandre gomesalexandre changed the title feat: toggle SUI feat: toggle SUI flag Dec 23, 2025
@gomesalexandre gomesalexandre marked this pull request as ready for review December 26, 2025 15:44
@gomesalexandre gomesalexandre requested a review from a team as a code owner December 26, 2025 15:44
@gomesalexandre
Copy link
Copy Markdown
Contributor Author

Yeeting after full test - to be retested in release.

@gomesalexandre gomesalexandre merged commit d18f511 into develop Dec 26, 2025
1 check passed
@gomesalexandre gomesalexandre deleted the claude/toggle-sui-flag-XGT9M branch December 26, 2025 15:44
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