Skip to content

Conversation

@collinr3
Copy link
Contributor

Merged upstream tweak for #53 and provided a solution to #52 and #16. Updated tests associated with #52 to reflect #53

@j4mie - I think I've got this change correct and complete now

System check identified no issues (0 silenced).
Ran 29 tests in 7.164s

OK
System check identified no issues (0 silenced).
.............................
----------------------------------------------------------------------
Ran 29 tests in 7.158s

OK
# Conflicts:
#	django_dbq/management/commands/worker.py
@j4mie
Copy link
Member

j4mie commented Jan 21, 2023

Sorry for the huge delay in responding to this. My hunch is that the approach taken in this PR is too complex. I've had a go at an alternative approach in #59 - what do you think?

@collinr3
Copy link
Contributor Author

collinr3 commented Jan 24, 2023

Hi @j4mie - thanks for looking into this. Comments added to #59

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.

2 participants