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

dev.ost.com Fixes | users/airdrop/drop param requirements #55

Closed
noslav opened this issue Apr 11, 2018 · 2 comments
Closed

dev.ost.com Fixes | users/airdrop/drop param requirements #55

noslav opened this issue Apr 11, 2018 · 2 comments

Comments

@noslav
Copy link
Contributor

noslav commented Apr 11, 2018

the input params are not declared mandatory or optional

@noslav noslav self-assigned this Apr 11, 2018
@noslav noslav changed the title dev.ost.com Fixes | users/airdrop dev.ost.com Fixes | users/airdrop param requirements Apr 11, 2018
@noslav
Copy link
Contributor Author

noslav commented Apr 20, 2018

  • error response msg to be corrected for errors.

{ "success": false, "err": { "code": "companyRestFulApi(s_am_s_3:HyEOVLDhz)", "msg": "Invalid airdropParams", "error_data": {} } }

  • method to check for the BT balance of the client

  • Check spelling in error messages
    { success: false, err: { code: 'companyRestFulApi(l_aa_sa_5:rJq6t8v2z)', msg: 'Invalid List type to airdrop users', error_data: [ [Object] ] } }

  • list_type is mandatory but not mentioned in the documentation as such

@noslav noslav changed the title dev.ost.com Fixes | users/airdrop param requirements dev.ost.com Fixes | users/airdrop/drop param requirements Apr 20, 2018
@noslav
Copy link
Contributor Author

noslav commented May 17, 2018

Fixed by #45

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant