Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Update models.py #369

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update models.py #369

wants to merge 1 commit into from

Conversation

codehero7386
Copy link

Brief Summary of Changes

What does this PR address?

  • GitHub issue (Add reference - #XX)
  • Refactoring
  • New feature
  • Bug fix
  • Adds more tests

Are tests included?

  • Yes
  • No

Reviewer, please note:

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I ran the full test suite before pushing the changes and all the tests pass.

@const-cloudinary
Copy link
Contributor

@codehero7386 thank you for contribution!
Could you explain a bit what are changes about?
Also it looks like all test failed, see for example:
https://app.travis-ci.com/github/cloudinary/pycloudinary/jobs/611413196
django.db.utils.OperationalError: no such column: django_tests_poll.created_at

Adding/updating some of those tests would be great!

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@codehero7386 @const-cloudinary