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
Backend tests for campaign middleware and sms campaign routes #1146
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.
Some minor suggestions and clean up required. Also, should we include the tests for the other channels (Email and Telegram) in this PR also?
@miazima could I get your help to resolve the merge conflicts? Thanks! |
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.
lgtm!
Tests:
GET /campaign/{id}/sms
PUT /campaign/{id}/sms/template
GET /campaign/{id}/sms/upload/start
POST /campaign/{id}/sms/upload/complete