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

Remove Brave Swap Fee #38566

Closed
3 of 6 tasks
StephenHeaps opened this issue May 24, 2024 · 5 comments · Fixed by brave/brave-core#23960
Closed
3 of 6 tasks

Remove Brave Swap Fee #38566

StephenHeaps opened this issue May 24, 2024 · 5 comments · Fixed by brave/brave-core#23960
Assignees
Labels
feature/web3/wallet/core feature/web3/wallet Integrating Ethereum+ wallet support OS/Android Fixes related to Android browser functionality OS/Desktop OS/iOS Fixes related to iOS browser functionality priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass - Android ARM QA Pass - iPhone QA Pass-Win64 QA/Yes release-notes/include

Comments

@StephenHeaps
Copy link

Description

We should remove Brave Fee for Swaps on all platforms. WebUI for desktop & Android, native for iOS. Implementation TBD.

Desktop & Android WebUI Swap iOS Swap

Steps to reproduce

  1. Fetch a price quote in swap
  2. Brave Fee label is shown with a percentage beside it

Actual result

Brave fee is shown for swaps (Ethereum & Solana)

Expected result

No Brave fee for swaps (Ethereum or Solana)

Reproduces how often

Easily reproduced

Desktop Brave version (brave://version info)

All

Android device

All

Channel information

  • release (stable)
  • beta
  • nightly

Reproducibility

  • with Brave Shields disabled
  • with Brave Rewards disabled
  • in the latest version of Chrome

Miscellaneous information

No response

@StephenHeaps StephenHeaps added OS/Android Fixes related to Android browser functionality OS/Desktop OS/iOS Fixes related to iOS browser functionality labels May 24, 2024
@jamesmudgett jamesmudgett added the priority/P2 A bad problem. We might uplift this to the next planned release. label May 31, 2024
@brave-builds brave-builds added feature/web3/wallet Integrating Ethereum+ wallet support feature/web3/wallet/core labels Jun 3, 2024
@brave-builds brave-builds added this to the 1.68.x - Nightly milestone Jun 3, 2024
@kjozwiak
Copy link
Member

kjozwiak commented Jun 4, 2024

The above requires 1.66.120 or higher for 1.66.x verification 👍

@srirambv
Copy link
Contributor

srirambv commented Jun 5, 2024

Verification passed on

Brave 1.66.120 Chromium: 125.0.6422.165 (Official Build) (64-bit)
Revision bd9a9589253e659c93a1c77eb9a3974525a80312
OS Windows 11 Version 23H2 (Build 22631.3672)
38566.mp4
@srirambv
Copy link
Contributor

srirambv commented Jun 5, 2024

Verification passed on Google Pixel 8 with Android 14 running 1.66.120 x64 build

38566.mp4
@srirambv
Copy link
Contributor

srirambv commented Jun 6, 2024

Verification passed on iPhone 13 running 1.66.1 (120)

38566.mp4
@kjozwiak kjozwiak removed this from the 1.66.x - Release #6 milestone Jun 11, 2024
@kjozwiak kjozwiak added this to the 1.67.x - Release milestone Jun 11, 2024
@kjozwiak
Copy link
Member

Moved this into the 1.67.x milestone as we're not going to be releasing another 1.66.x with C125. We'll be releasing C126 via 1.67.x on June 13th. Added so it can be included in the 1.67.x release notes.

However, keep in mind that the above was released on Android via 1.66.x - Release #6 so there's no need to add the above into the release notes for Android. CCing @hffvld.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/web3/wallet/core feature/web3/wallet Integrating Ethereum+ wallet support OS/Android Fixes related to Android browser functionality OS/Desktop OS/iOS Fixes related to iOS browser functionality priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass - Android ARM QA Pass - iPhone QA Pass-Win64 QA/Yes release-notes/include
7 participants