License change #222
Merged
License change #222
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
There was a problem hiding this comment.
Pull Request Overview
This pull request updates the licensing across the entire Reown Flutter SDK ecosystem from Apache 2.0 to proprietary community license agreements. The change introduces usage-based commercial licensing thresholds while maintaining free use for educational and small-scale commercial applications.
- Replaces Apache 2.0 with WalletConnect Community License Agreement for wallet-related packages
- Introduces Reown Community License Agreement for app-focused packages
- Establishes commercial licensing requirements based on RPC calls (2.5M monthly) and MAU limits (500 monthly)
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| LICENSE | Updated to WalletConnect Community License Agreement with commercial usage thresholds |
| packages/reown_yttrium/LICENSE | Changed from Apache 2.0 to WalletConnect Community License |
| packages/reown_walletkit/LICENSE | Changed from Apache 2.0 to WalletConnect Community License |
| packages/reown_sign/LICENSE | Changed from Apache 2.0 to WalletConnect Community License |
| packages/reown_core/LICENSE | Changed from Apache 2.0 to WalletConnect Community License |
| packages/reown_cli/LICENSE | Changed from Apache 2.0 to Reown Community License |
| packages/reown_appkit/LICENSE | Changed from Apache 2.0 to Reown Community License |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
arein
approved these changes
Aug 20, 2025
quetool
added a commit
that referenced
this pull request
Aug 20, 2025
* Solflare wallet support * added secure storage * added secure storage * fix conflicts * solflare fix * Update Flutter SDK to latest (3.35.1), freezed and freezed_annotation * dart style * Update flutter version on CI * Android app_plugin_loader migration * fixes * Upgrade versions and changelogs * License change (#222) * License change * change file extension * change license file extensions
quetool
added a commit
that referenced
this pull request
Oct 16, 2025
* rollback * Chores/refactor (#220) * First refactor * Second refactor * minor change * Solflare wallet support (#219) * Solflare wallet support * added secure storage * added secure storage * fix conflicts * solflare fix * Update Flutter SDK to latest (3.35.1), freezed and freezed_annotation * dart style * Update flutter version on CI * Android app_plugin_loader migration * fixes * Upgrade versions and changelogs * License change (#222) * License change * change file extension * change license file extensions * Fix force push block (#223) * license * minor change * Flutter publish fixes (#224) * fix for flutter publish * license format change * [Chore] publish workflow creates PR instead of pushing to develop (#214) * publish workflow creates PR * change to use GITHUB_TOKEN * Update .github/workflows/publish-packages.yml Co-authored-by: Copilot <[email protected]> --------- Co-authored-by: Copilot <[email protected]> * Update dependencies (#226) * Minor change (#225) * cta assistant (#234) * cta assistant * advanced * Update .github/workflows/cta.yml Co-authored-by: Ben Kremer <[email protected]> * remove unused config --------- Co-authored-by: Ben Kremer <[email protected]> * TVF data structure fix and other bug fixes (#232) * wallet_sendCalls 2.0.0 * Tests * fix tests * added try catch to maintain consistence with other methods * minor change * version updates * TVF data structure fix + test * tvf for wallet_sendCalls 2.0.0 * minor change * minor change * Change in workflow (#235) * Change in workflow * Fix with AI * change on commits detection * fix (#236) * [WalletKit] Remove Yttrium dependency (#245) * removing yttrium * reverse some changes * Remove comments * Bump rexml from 3.3.9 to 3.4.2 in /packages/reown_walletkit/example/ios (#240) Bumps [rexml](https://github.com/ruby/rexml) from 3.3.9 to 3.4.2. - [Release notes](https://github.com/ruby/rexml/releases) - [Changelog](https://github.com/ruby/rexml/blob/master/NEWS.md) - [Commits](ruby/rexml@v3.3.9...v3.4.2) --- updated-dependencies: - dependency-name: rexml dependency-version: 3.4.2 dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * [NEW SDK] reown_pos 0.0.1 with demo (#228) * reown_pos 0.0.1 with demo * reafctor * readme file * structure changes * minor change * minor change * refactor * Refactor and fixes * minor change * minor change * final fixes * minor change * fix for web * fixes and enhancements * change to new api * final changes * minor change * [reown_pos] New methods names and structures (#233) * methods names and structure changes * rollback * rollback * minor fix * cta assistant (#234) * cta assistant * advanced * Update .github/workflows/cta.yml Co-authored-by: Ben Kremer <[email protected]> * remove unused config --------- Co-authored-by: Ben Kremer <[email protected]> * TVF data structure fix and other bug fixes (#232) * wallet_sendCalls 2.0.0 * Tests * fix tests * added try catch to maintain consistence with other methods * minor change * version updates * TVF data structure fix + test * tvf for wallet_sendCalls 2.0.0 * minor change * minor change * Change in workflow (#235) * Change in workflow * Fix with AI * change on commits detection * fix (#236) --------- Co-authored-by: Ben Kremer <[email protected]> * [reown_pos] enhancements (#237) * methods names and structure changes * rollback * rollback * minor fix * pos changes 1b * pos changes 1b * cta assistant (#234) * cta assistant * advanced * Update .github/workflows/cta.yml Co-authored-by: Ben Kremer <[email protected]> * remove unused config --------- Co-authored-by: Ben Kremer <[email protected]> * TVF data structure fix and other bug fixes (#232) * wallet_sendCalls 2.0.0 * Tests * fix tests * added try catch to maintain consistence with other methods * minor change * version updates * TVF data structure fix + test * tvf for wallet_sendCalls 2.0.0 * minor change * minor change * Change in workflow (#235) * Change in workflow * Fix with AI * change on commits detection * fix (#236) * posSupportedNetworks implementation * minor change * Minor changes * Error handling * Minor change * minor change * minor change * Minor changes * Minor change * better errors and tests * New error codes from BAPI * comment * comment * bug fixes * Minor changes * minor change --------- Co-authored-by: Ben Kremer <[email protected]> * Fix for binance wallet (#242) * [reown_pos] minor improvements (#238) * Minor improvements * Restart on ConnectFailedEvent * minor change * change name service * minor changes * minor change * delete unsupported platforms * Second binance fix (#243) * Second binance fix * revert * change ref * Created new package with new pos_client name (#244) * Created new package with new pos_client name * minor change * minor change * adding pos_client step * Minor changes * bug fixes on sample * change ref * Remove reown_pos folder in favor of pos_client * version changes * regenerate --------- Co-authored-by: Ben Kremer <[email protected]> * Minor change (#255) * chore: updated sample apps certificates (#254) Co-authored-by: Alfreedom <[email protected]> * Verify V2 (#257) * Verify V2 * regenerate * Tests * minor changes * version generation * Increase the maximum heap size * changelog * Update versions (#262) * Bugfix/verify v2 bugfix (#265) * verify v2 bugfix * Fix for issue 252 (amends PR 247) (#258) * Fix for issue 252 (amends PR 247) * Update packages/reown_core/lib/relay_client/websocket/websocket_handler.dart Co-authored-by: Copilot <[email protected]> * Applying null-safety suggestions --------- Co-authored-by: Copilot <[email protected]> * fixes on sample wallet * minor change --------- Co-authored-by: Copilot <[email protected]> * chore: configure Claude auto-review action (#266) * chore: configure Claude auto-review action Co-Authored-By: Ben Kremer <[email protected]> * chore: enhance Claude auto-review project context Co-Authored-By: Ben Kremer <[email protected]> * chore: update reown_cli description in project context Co-Authored-By: Ben Kremer <[email protected]> * chore: update pos_client description in project context Co-Authored-By: Ben Kremer <[email protected]> --------- Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: Ben Kremer <[email protected]> * TON support on Wallet Sample (#267) * Ton support on WalletKit side through Yttrium * update and generate files * Feature/ton support (#261) * First steps * changes * Ton implementation on Android * ios supported * minor changes and rollbacks * revert * dart format * fix * Feature/ton sui stacks (#271) * Added SUI and Stacks * refactor * minor fixes --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Copilot <[email protected]> Co-authored-by: Ben Kremer <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Ignacio Santise <[email protected]> Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: Ben Kremer <[email protected]>
|
I’m genuinely disappointed to see WalletConnect switch from Apache‑2.0 to a proprietary license. WalletConnect has been a critical FOSS bridge between wallets and dApps, and this change will sway many developers, myself included, and projects to stop using it. Making it proprietary may provide short-term control, but it undermines the Web3 ecosystem WalletConnect has helped build. I hope the team considers the long-term impact on adoption and loss in developer trust—this was a key piece of open infrastructure for Web3, and restricting it could push the community toward inevitable alternatives. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
LICENSE changes
How Has This Been Tested?
Due Dilligence