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

[Desktop] Claim doesn't work for existing wallets #11144

Closed
NejcZdovc opened this issue Aug 10, 2020 · 2 comments · Fixed by brave/brave-core#6384
Closed

[Desktop] Claim doesn't work for existing wallets #11144

NejcZdovc opened this issue Aug 10, 2020 · 2 comments · Fixed by brave/brave-core#6384

Comments

@NejcZdovc
Copy link
Contributor

Description

When you are connecting already used KYC wallet to claim it will return 200 compared to 201 for new wallets. We need to handled 200 response as well.

Steps to Reproduce

  1. Enable rewards
  2. Connect old KYC wallet that was already connected

Actual result:

Claim fails

Expected result:

Claim should work

Reproduces how often:

Brave version (brave://version info)

Version/Channel Information:

  • Can you reproduce this issue with the current release?
  • Can you reproduce this issue with the beta channel?
  • Can you reproduce this issue with the nightly channel?

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields?
  • Does the issue resolve itself when disabling Brave Rewards?
  • Is the issue reproducible on the latest version of Chrome?

Miscellaneous Information:

@LaurenWags
Copy link
Member

Uplift approved, waiting on new 1.13.x build to test.

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Aug 26, 2020

Verification passed on


Brave | 1.13.80 Chromium: 85.0.4183.83 (Official Build) (64-bit)
-- | --
Revision | 94abc2237ae0c9a4cb5f035431c8adfb94324633-refs/branch-heads/4183@{#1658}
OS | Windows 10 OS Version 1903 (Build 18362.1016)

[ REQUEST ]
> URL: https://grant.rewards.bravesoftware.com/v3/wallet/uphold/redacted/claim
> Method: UrlMethod::POST
> Content: ...
[ RESPONSE - OnTransferFunds ]
> Url: https://grant.rewards.bravesoftware.com/v3/wallet/uphold/redacted/claim
> Result: Success
> HTTP Code: 200
[ REQUEST ]
> URL: https://grant.rewards.bravesoftware.com/v1/suggestions/claim
> Method: UrlMethod::POST
> Content:
[ RESPONSE - OnRedeemTokens ]
> Url: https://grant.rewards.bravesoftware.com/v1/suggestions/claim
> Result: Success
> HTTP Code: 200
  • Verified Ad grants are moved to uphold card and wallet status is displayed as Verified in brave://rewards-internals
    image
    image
    image
  • Verified one-time tips are processed successfully
  • Verified recurring tips are processed successfully
  • Verified AC is processed successfully

Verification PASSED on macOS 10.15.6 x64 using the following builds:

Brave | 1.13.80 Chromium: 85.0.4183.83 (Official Build) (64-bit)
-- | --
Revision | 94abc2237ae0c9a4cb5f035431c8adfb94324633-refs/branch-heads/4183@{#1658}
OS | macOS Version 10.15.6 (Build 19G73)

Went through the STR/Cases outlined via brave/brave-core#6384 and brave/brave-core#6401.

Before Connecting Uphold Before Connecting Uphold
Screen Shot 2020-08-26 at 12 48 20 PM Screen Shot 2020-08-26 at 12 48 36 PM

Connected to a KYC staging account and ensured the following logs appeared without any issues:

[ REQUEST ]
> URL: https://grant.rewards.bravesoftware.com/v3/wallet/uphold/[..............]/claim
> Method: UrlMethod::POST
> Content....
[ RESPONSE - OnTransferFunds ]
> Url: https://grant.rewards.bravesoftware.com/v3/wallet/uphold/[..............]/claim
> Result: Success
> HTTP Code: 200
[ REQUEST ]
> URL: https://grant.rewards.bravesoftware.com/v1/suggestions/claim
> Method: UrlMethod::POST
> Content...
[ RESPONSE - OnRedeemTokens ]
> Url: https://grant.rewards.bravesoftware.com/v1/suggestions/claim
> Result: Success
> HTTP Code: 200

Once claimed, ensured that brave://rewards and brave://rewards-internals displayed the correct information:

After Connecting Uphold After Connecting Uphold
Screen Shot 2020-08-26 at 12 49 43 PM Screen Shot 2020-08-26 at 12 50 25 PM

Ensured that it also appeared correctly in Uphold as per the following:

Screen Shot 2020-08-26 at 12 50 44 PM

Ensured that you can do the following:

  • Tip verified publishers
  • ensured that a-c is being processed
  • ensured that recurring tips are being processed during a-c

Verification passed on

Brave 1.13.80 Chromium: 85.0.4183.83 (Official Build) (64-bit)
Revision 94abc2237ae0c9a4cb5f035431c8adfb94324633-refs/branch-heads/4183@{#1658}
OS Ubuntu 18.04 LTS

Verified STR outlined via brave/brave-core#6384 and brave/brave-core#6401.

Before connecting to uphold

image image

Connected to a KYC staging account and ensured the following logs appeared without any issues:

[ RESPONSE - OnTransferFunds ]
> Url: https://grant.rewards.bravesoftware.com/v3/wallet/uphold/[redacted]/claim
> Result: Success
> HTTP Code: 200
> Body: null

[ RESPONSE - OnRedeemTokens ]
> Url: https://grant.rewards.bravesoftware.com/v1/suggestions/claim
> Result: Success
> HTTP Code: 200


Once claimed, ensured that brave://rewards and brave://rewards-internals displayed the correct information:

image image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment