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

RDR 1358 remove suits #19

Merged
merged 3 commits into from
Jul 4, 2016
Merged

RDR 1358 remove suits #19

merged 3 commits into from
Jul 4, 2016

Conversation

aahunter
Copy link
Contributor

@aahunter aahunter commented Jul 1, 2016

suits isn't installed but lingering references remain, removed references

@id2359
Copy link
Member

id2359 commented Jul 4, 2016

sounds like a good idea - just fyi we do still use the admin interface though

@aahunter
Copy link
Contributor Author

aahunter commented Jul 4, 2016

@id2359 Can you point me to somewhere these changes might be a problem?

@id2359
Copy link
Member

id2359 commented Jul 4, 2016

Looks like Maciej has cleaned up pages linked from Admin Pages but you could try http://localhost:8000/admin/groups/customuser/ for a test - I can't find suit referenced on the pages I've tested locallly just now so should be ok.

@id2359
Copy link
Member

id2359 commented Jul 4, 2016

only the error pages were using ( which you fixed) - so ok

@aahunter aahunter merged commit c6ce327 into next_release Jul 4, 2016
@aahunter aahunter deleted the rdr-1358-remove-suits branch July 5, 2016 01:09
id2359 pushed a commit that referenced this pull request Jul 7, 2016
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