Skip to content

[UPDATE] Store coin auto payout in coin_addresses table per coin #2404

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

Merged
merged 5 commits into from
Apr 16, 2015

Conversation

TheSerapher
Copy link
Contributor

Quick update on the way we store the Auto Payout Threshold for accounts:

  • Added ap_threshold to coin_addresses table
  • Migrate users settings from accounts table to coin_addresses table for the current MPOS instance
  • Delete ap_threshold from accounts table
  • Update base SQL structure for new instances

Also update some classes to reflect this change, but this needs some testing before I will merge this.

@TheSerapher
Copy link
Contributor Author

Release this branch to my Fluttercoin test pool and migration worked (after fixing a copy and paste error in the upgrade SQL). I will now let this run until the next AP payout is triggered to see if payouts are still working.

If all is good, I merge into development for the 1.0.2 release.

@TheSerapher TheSerapher added this to the v1.0.2 milestone Apr 16, 2015
@TheSerapher TheSerapher self-assigned this Apr 16, 2015
@scorpio18
Copy link

I just test it and is working , the mpos is now updates and reads the ap_threshold from coin_addresses table , but for the moment i did not test it if the payout is triggered on AP , i will try in a 1 -2 hours , i need to find a coin with low diff to test it

@TheSerapher
Copy link
Contributor Author

Thanks @scorpio18 , I am also running it on a pool right now and will see if I run into any issues. Looks pretty good so far, auto-payouts seem to work. I have asked a friend to test the manual payout to make sure it all works as expected but will merge into development if you give me a second okay on this.

@TheSerapher
Copy link
Contributor Author

Just got the OK, manual Payouts also work.

@scorpio18
Copy link

I'm just testing myself now , but will take some time until the blocks get confirmed so i can make a AP and manual payment

@scorpio18
Copy link

Testing done , manual payouts and ap working just fine

TheSerapher added a commit that referenced this pull request Apr 16, 2015
[UPDATE] Store coin auto payout in coin_addresses table per coin
@TheSerapher TheSerapher merged commit 76a2853 into development Apr 16, 2015
@TheSerapher TheSerapher deleted the coin-ap-threshold branch April 16, 2015 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants