-
Notifications
You must be signed in to change notification settings - Fork 107
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 support for chunked uploads #18
Conversation
750b7d5
to
50749ea
Compare
fb1b8c1
to
701deed
Compare
Codecov Report
@@ Coverage Diff @@
## master #18 +/- ##
==========================================
- Coverage 76.2% 74.52% -1.69%
==========================================
Files 64 65 +1
Lines 2824 2865 +41
==========================================
- Hits 2152 2135 -17
- Misses 672 730 +58
Continue to review full report at Codecov.
|
validated_data (dict): Data to save to the database | ||
""" | ||
artifact = super().create(validated_data) | ||
if hasattr(self, 'upload'): |
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 would think that the upload attribute is always there. However it's value is None or HyperlinkedRelatedField. Is my understanding correct? In that case I would check if self.upload is None or not.
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.
No, the upload attribute gets optionally created in the validate method. See line 173 above.
fixes #4196
https://pulp.plan.io/issues/4196
Please be sure you have read our documentation on creating PRs:
https://docs.pulpproject.org/en/3.0/nightly/contributing/pull-request-walkthrough.html