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

Active subscription not listed under "active subscriptions" #3763

Closed
pascalvanhecke opened this issue Feb 17, 2019 · 21 comments
Closed

Active subscription not listed under "active subscriptions" #3763

pascalvanhecke opened this issue Feb 17, 2019 · 21 comments
Labels
bug This is something that isn't working as intended. Gitcoin Grants Gitcoin Grants

Comments

@pascalvanhecke
Copy link

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 #3762 and possibly has the same cause (failed and/or overridden cancellations being interpreted as actual cancellations)

@PixelantDesign
Copy link
Contributor

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

@PixelantDesign PixelantDesign added the bug This is something that isn't working as intended. label Feb 18, 2019
@frankchen07 frankchen07 added the Gitcoin Grants Gitcoin Grants label Feb 18, 2019
@frankchen07 frankchen07 added the 3 label Feb 20, 2019
@frankchen07 frankchen07 assigned owocki and unassigned danlipert Feb 21, 2019
@frankchen07 frankchen07 removed the 3 label Feb 28, 2019
@gitcoinbot
Copy link
Member

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.

@MatthewPChapdelaine
Copy link

Another step would be getting Docker installed for testing.

@MatthewPChapdelaine
Copy link

Docker installed.

@MatthewPChapdelaine
Copy link

new project

@frankchen07
Copy link
Contributor

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

@frankchen07
Copy link
Contributor

frankchen07 commented Mar 21, 2019

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.

@frankchen07
Copy link
Contributor

@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.

@frankchen07
Copy link
Contributor

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

@speedyFixer
Copy link
Contributor

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

@speedyFixer
Copy link
Contributor

speedyFixer commented Apr 24, 2019

@frankchen07 @gitcoinbot right now working on this!

Updated Plan status:

  • Prepare a local environment for debugging
  • Reading the repository
  • Localization of subscription related code
  • 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

@speedyFixer
Copy link
Contributor

Status Update:

I found some times num_tx_approved is missing in POST request and the default value assigned to It is an int:

subscription.num_tx_approved = request.POST.get('num_tx_approved', 1)

Therefore, in the following condition the result will be False:

if subscription.num_tx_approved == '1':

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)

@owocki
Copy link
Contributor

owocki commented Apr 25, 2019

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

@speedyFixer
Copy link
Contributor

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.

@speedyFixer
Copy link
Contributor

speedyFixer commented Apr 25, 2019

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

@frankchen07
Copy link
Contributor

I have not - @ceresstation - have you?

@speedyFixer
Copy link
Contributor

Hi @ceresstation , any update?

@speedyFixer
Copy link
Contributor

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

@frankchen07
Copy link
Contributor

closing this out until the bug resurfaces again

@gitcoinbot
Copy link
Member

⚡️ 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This is something that isn't working as intended. Gitcoin Grants Gitcoin Grants
Projects
None yet
Development

No branches or pull requests

8 participants