Skip to content
This repository has been archived by the owner on Sep 17, 2019. It is now read-only.

Bugfix/clean messages #93

Merged
merged 2 commits into from
Mar 18, 2017
Merged

Conversation

itcrab
Copy link
Contributor

@itcrab itcrab commented Mar 18, 2017

What's this Pull Request do?

>python manage.py clear_sent_messages 20
    MailerMessage.objects.clear_sent_messages(offset=options['offset'])
  File "\lib\site-packages\mailqueue\models.py", line 35, in clear_sent_messages
    delete_before = timezone.now() - offset
TypeError: unsupported operand type(s) for -: 'datetime.datetime' and 'list'

What has been changed?

  • Change base type for "offset" argument in command to int (was list)

Any background context you want to provide?

Screenshots

GitHub issues

I don't create any issues and in current issues no see this problem.

@dstegelman
Copy link
Owner

Going to merge this, and also remove the positional argument to make it the same as the other management command.

@dstegelman dstegelman merged commit 4f31ce6 into dstegelman:master Mar 18, 2017
@dstegelman
Copy link
Owner

Released in 3.0.2

@itcrab
Copy link
Contributor Author

itcrab commented Mar 19, 2017

@dstegelman thanks for speed))

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants