-
-
Notifications
You must be signed in to change notification settings - Fork 30
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
Add SMSPool #184
Add SMSPool #184
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall a good start. I'll approve checks and look it over once some changes are made.
Cheers for the input, I will work on it now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the wait and thanks for your patience. Just a few more things (and making sure the tests pass) and then I can get this merged for you (no new comments, just add-ons to the ones above).
- Updated description to use more neutral wording - Adjusted score for security - Added CF to noncritical
Waiting for checks to pass, but will merge shortly after. Thanks ! |
Not a problem, thank you so much. |
Type of pull request: product addition
Related issues: list the related issues here (if applicable)
Privacy policy: https://www.smspool.net/privacy-policy
Terms of service: https://www.smspool.net/tos
Provide more information about your pull request here. For help, see our contribution guide: http://privacyspy.org/contribute/