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

Some users can't receive coins in IAP #9

Open
immartian opened this issue May 21, 2019 · 6 comments
Open

Some users can't receive coins in IAP #9

immartian opened this issue May 21, 2019 · 6 comments
Assignees
Labels
bug Something isn't working

Comments

@immartian
Copy link
Contributor

immartian commented May 21, 2019

There are several users can't receive coins in their account after newly creating an account in the app, yet their money in Apple account would be deducted. Two reasons emerged: 1. empty wallet; 2. transaction failure. cc @riverfor @bengyles

@immartian
Copy link
Contributor Author

it's related to the backend, but the client should have protection. https://github.com/Musicoin/api/issues/22

@bengyles
Copy link
Collaborator

I propose we block the IAP when there is not wallet available in the app and show a message there is no wallet address available in the wallet screen for now. I think it's also a good idea to hide the "$Buy MUSIC" option as well when this issue occurs.

When we have a solution in the backend we can add a button to "solve" the wallet issue in case we need that. What do you think @immartian @riverfor ?

@immartian
Copy link
Contributor Author

+1
It can be a hotfix as step 1, right?
so step 2, if I'm not wrong, is to give users an option to fix it by clicking a link("get a new wallet"), likely?

@bengyles
Copy link
Collaborator

@immartian Yes indeed, step 1 is a hotfix and can be pushed over the air. Step 2 can also be done over the air but needs some more development, we can add it as a hotfix too or include it in the next version. If we could just fix it in the backend without having to add a button or if they can just logout and login again to fix the problem that would be better ofcourse.

@immartian
Copy link
Contributor Author

We need both of them implemented asap as IAP is the crucial line of us on user's satisfaction and our own income. Please have a hotfix first then integrate backend solution to have an inhere resolve.

@riverfor
Copy link

READY FOR TEST on t2.
@bengyles @immartian

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants