-
-
Notifications
You must be signed in to change notification settings - Fork 177
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
Code refactoring - Replace shadow models imports with their openrosa app counterparts #5037
Code refactoring - Replace shadow models imports with their openrosa app counterparts #5037
Conversation
@@ -429,7 +431,7 @@ def test_google_transcript_post(self, m1, m2): | |||
'submission': submission_id, | |||
'q1': {GOOGLETS: {'status': 'requested', 'languageCode': ''}} | |||
} | |||
with self.assertNumQueries(FuzzyInt(51, 57)): | |||
with self.assertNumQueries(FuzzyInt(210, 215)): |
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.
We discussed that this only affects tests and is temporary, due to creating XForm
s on the fly:
https://github.com/kobotoolbox/kpi/pull/5037/files#diff-3c6227c824c76da07fffa9c1cae3296a272e63c4eb7a633b2795a8d62e207a02R727
@@ -38,39 +36,6 @@ | |||
) | |||
|
|||
|
|||
def update_autofield_sequence(model): |
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.
This is still called by KobocatUser.sync()
in this file, but it's not imported from its new location kpi/utils/database.py
.
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.
Notes
Part I of #5036. Remove shadow models imports everywhere except in
KobocatDeploymentBackend
Related issues
#5036