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

Referral Program #188

Open
2 of 16 tasks
atlantic-blockchain opened this issue Oct 17, 2019 · 2 comments
Open
2 of 16 tasks

Referral Program #188

atlantic-blockchain opened this issue Oct 17, 2019 · 2 comments
Labels
bug Something isn't working help wanted Extra attention is needed question Further information is requested

Comments

@atlantic-blockchain
Copy link

atlantic-blockchain commented Oct 17, 2019

Bug Description

Referral program may not being working as intended. Filing as bug, until the time where it is verified the referral program is working as designed.

Within the following file, /peerplays/libraries/chain/affiliate_payout.cpp there is no indication as to whether or not the users affiliate account is receiving the intended 20% of the rake.

The following file peerplays/libraries/chain/db_bet.cpp dictates which account will receive the 20% affiliate payout. It appears that the account that WON, is receiving the 20% payout, which means the funds are staying in the account that won.

Impacts

The peerplays referral program is impacted by this bug. If the funds never move from the winners hands, to the affiliates hands, then there is no incentive for anyone to be an affiliate partner for the peerplays blockchain.

Steps To Reproduce

  • Create an account (account-one)
  • Upgrade account-one
  • Create a second account (account-two) and use (account-one) as the referrer
  • Create a third account (account-three)
  • Give 10PPY to both account-two and account-three
  • Have account 2 place two bets, one on each side of the market, which will guarantee a win
  • Have account 3 take account-two's bets. Ensuring that account-two will win at least one bet
  • Upon resolution of the markets, check to see that account-one has received 20% of 1% of account-two's winnings

Expected Behavior

Upon resolution of the markets, check to see that account-one has received 20% of 1% of account-two's winnings

Impacts

  • API (the application programming interface)
  • Build (the build process or something prior to compiled code)
  • CLI (the command line wallet)
  • Deployment (the deployment process after building such as Docker, Gitlab, etc.)
  • P2P (the peer-to-peer network for transaction/block propagation)
  • Performance (system or user efficiency, etc.)
  • Protocol (the blockchain logic, consensus, validation, etc.)
  • Security (the security of system or user data, etc.)
  • UX (the User Experience)
  • Other (please add below)

PBSA / Developer tasks

  • Evaluate / Prioritize Bug Report
  • Refine User Stories / Requirements
  • Define Test Cases
  • Design / Develop Solution
  • Perform QA/Testing
  • Update Documentation
@bobinson
Copy link

@atlantic-blockchain - This is reported against the current mainnet or TESTNET ?

@bobinson bobinson added the question Further information is requested label Oct 17, 2019
@atlantic-blockchain
Copy link
Author

@atlantic-blockchain - This is reported against the current mainnet or TESTNET ?

MainNet

@bobinson bobinson added bug Something isn't working help wanted Extra attention is needed labels Oct 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants