Web: Active subscription not listed under "active subscriptions"

Created on 17 Feb 2019  路  21Comments  路  Source: gitcoinco/web

uniswap gitcoin 2019-02-17 14-15-36

Describe the bug

user pascalvanhecke has active subscription (as indicated by interface in screenshot) but subscription is not listed as active. Also: the subscription has not been charged after 2 days.

Expected behavior
Subscription should have been charged (it was intended as a one-time donation), and should have been registered as a contribution

Additional context

The bug is probably related to an earlier reported bug https://github.com/gitcoinco/web/issues/3762 and possibly has the same cause (failed and/or overridden cancellations being interpreted as actual cancellations)

Gitcoin Grants bug

All 21 comments

Thanks for the report @pascalvanhecke! We'll get this prioritized asap!
cc @frankchen07

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


__This issue now has a funding of 0.33 ETH (45.5 USD @ $137.89/ETH) attached to it as part of the gitcoinco fund.__

Another step would be getting Docker installed for testing.

Docker installed.

new project

@Razorblade16 - sounds good, looking forward to seeing your work!

hi @MatthewPChapdelaine , any progress here? if not, I will remove you from this bounty and release back to the public. If you decide you want to try again, feel free to start work again.

@MatthewPChapdelaine - I'm going to remove you from this bounty since you've been non-responsive. If you come back and the bounty is still open, feel free to reach out and communicate on any progress.

@speedyfixer - sounds good! The issue includes fixing the bug.

@frankchen07 Perfect! @gitcoinbot as I wrote, I going to start tomorrow.

@frankchen07 @gitcoinbot right now working on this!

Updated Plan status:

  • [x] Prepare a local environment for debugging
  • [x] Reading the repository
  • [x] Localization of subscription related code
  • [x] Load grant data in local test environment
  • [ ] Describe a group of steps for reproduce the bug _Working in this item_
  • [ ] Fix the bug
  • [ ] Test the solution

Status Update:

I found some times num_tx_approved is missing in POST request and the default value assigned to It is an int: https://github.com/gitcoinco/web/blob/9ea8fbeaf0f545d210f3362d35239cbc2244552b/app/grants/views.py#L427

Therefore, in the following condition the result will be False: https://github.com/gitcoinco/web/blob/9ea8fbeaf0f545d210f3362d35239cbc2244552b/app/grants/views.py#L434

That behavior will create an Inactive subscription, please check the image:

Right records of contributions are with the code fix and inactive subscriptions is with the bug.

test_2

  • _What is the cause of the num_tx_approved be missing in the POST request?_
    Seems something with MetaMask in the JS code. (I'm not sure about It)

hmm that post should not happen without num_tx_approved in the POST..

Yes I'm aware of that, I'm looking deeper for the cause of missing num_tx_approved or check if I found another reason.

@owocki, @frankchen07, @PixelantDesign Are there more reports of users experiment this issue?

I have not - @ceresstation - have you?

Hi @ceresstation , any update?

I'm going to stop in Gitcoin while I wait for more info for fix this issue.

closing this out until the bug resurfaces again

鈿★笍 A tip worth 0.06000 ETH (14.63 USD @ $243.78/ETH) has been granted to @speedyfixer for this issue from @frankchen07. 鈿★笍

Nice work @speedyfixer! Your tip has automatically been deposited in the ETH address we have on file.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

ghost picture ghost  路  3Comments

wizzfile picture wizzfile  路  3Comments

abitrolly picture abitrolly  路  4Comments

jasonrhaas picture jasonrhaas  路  4Comments

abitrolly picture abitrolly  路  4Comments