Skip to content
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

Fixing issue 11 #14

Merged
merged 1 commit into from
May 24, 2017
Merged

Fixing issue 11 #14

merged 1 commit into from
May 24, 2017

Conversation

seg1129
Copy link
Collaborator

@seg1129 seg1129 commented May 18, 2017

No description provided.

@seg1129 seg1129 requested a review from tjrivera May 18, 2017 02:56
@tjrivera
Copy link
Contributor

@seg1129, thanks for your work on this. Can you provide a little background on this PR? I see two issues that I think are related (but not quite dupes) #11 and #12. I assume since this is against #11 that the intent is simply to fix the error that this was originally throwing Related Field has invalid lookup: icontains but doesn't necessarily make the admin search functionality usable (which I think is due to be addressed in #12?) Can we change the title of these issues to be more specific?

@seg1129 seg1129 merged commit d21133a into dev May 24, 2017
@seg1129 seg1129 mentioned this pull request May 24, 2017
@tjrivera tjrivera deleted the issue11-fix_admin_search branch May 24, 2017 18:14
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