CommunityCellularManager
CommunityCellularManager copied to clipboard
Network balance limit (Client dependent)
Please review network/subscriber credit limit implementation which include the below features:
- Cloud admin and admin have rights to set the highest denomination bracket (Reload limit) correspond to which subscriber can do top up.
- Cloud admin and admin have rights to set the maximum balance limit a subscriber can have. Retailer cannot transfer more than the set balance limit amount to any subscriber.
- Subscribers can not have the account balance greater than the set balance limit.
- Cloud admin and admin have rights to set the limit on maximum number of consecutive unsuccessful transactions.
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at [email protected]. Thanks!
If you are contributing on behalf of someone else (eg your employer): the individual CLA is not sufficient - use https://developers.facebook.com/opensource/cla?type=company instead. Contact [email protected] if you have any questions.
Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!
@shivkumarsah updated the pull request - view changes
Careful to split out reviews. I think this also contained #39
@shivkumarsah updated the pull request - view changes
@shivkumarsah updated the pull request - view changes
@shivkumarsah updated the pull request - view changes
is this ready for review?
yes , please review.As suggested comments are incorporated.
@shivkumarsah updated the pull request - view changes
@shivkumarsah updated the pull request - view changes
@9muir has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Please address comments from our internal linter:
If you rebase against the current head then three of the F401 errors should go away (I fixed them in a commit recently).
@shivkumarsah updated the pull request - view changes - changes since last import
@9muir has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.