-
Notifications
You must be signed in to change notification settings - Fork 4
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
Feature/mcmr 152 add channel from rp contact to registration #321
Feature/mcmr 152 add channel from rp contact to registration #321
Conversation
…act-to-registration
…ation' of github.com:praekeltfoundation/ndoh-hub into feature/MCMR-152-add-channel-from-rp-contact-to-registration
@@ -164,7 +164,7 @@ def string_field(field, value): | |||
VALUES %s | |||
ON CONFLICT DO NOTHING | |||
""", | |||
((contact_id, f"whatsapp:{wa}", wa, None, "whatsapp", ORG, 50, None, None),), | |||
((contact_id, f"whatsapp:{wa}", wa, None, "whatsapp", 1, 50, None, None),), |
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.
I think this change is unrelated, and something we want to keep. Try merging in develop
into this branch, and see if that fixes it.
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.
okay, pulled the develop version. But there is a flake8 error with that line being too long so I'll push those changes
In this PR, we add the contact field to missing registrations