Skip to content

Conversation

prathamesh920
Copy link
Contributor

For creating users, did not add bulk create as we need the created user
for creating profile and to add in the course.
So the course and profile is linked with the user. And bulk create does
not return the primary key, so becomes complex to create profile or add
user to the course.

For  creating users, did not add bulk create as we need the created user
for creating profile and to add in the course.
So the course and profile is linked with the user. And bulk create does
not return the primary key, so becomes complex to create profile or add
user to the course.
@pep8speaks
Copy link

Hello @prathamesh920! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 2498:13: E128 continuation line under-indented for visual indent
Line 2508:80: E501 line too long (80 > 79 characters)
Line 2510:80: E501 line too long (83 > 79 characters)

@codecov-io
Copy link

Codecov Report

Merging #838 (34c74a8) into master (4756f86) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #838   +/-   ##
=======================================
  Coverage   76.69%   76.69%           
=======================================
  Files          53       53           
  Lines        7684     7689    +5     
  Branches     1166     1166           
=======================================
+ Hits         5893     5897    +4     
- Misses       1408     1409    +1     
  Partials      383      383           
Impacted Files Coverage Δ
yaksh/views.py 70.73% <100.00%> (+0.12%) ⬆️
api/views.py 75.83% <0.00%> (-0.91%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e53b86c...34c74a8. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants