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

platform creation bug in usage module (was #36 in non merged repo) #4

Closed
wants to merge 16 commits into from

Conversation

PaulPoulain
Copy link
Member

fixes a bug that appeared in a November commit that creates a new Platform for every title.

@benheet
Copy link
Contributor

benheet commented Oct 18, 2016

creating new platforms for every title sounds like a significant problem. is this problem more limited in scope than what it sounds? If not then this might need to be a priority pull request.

@jeffnm
Copy link
Member

jeffnm commented Oct 18, 2016

I think we can close this request. It looks like the fixes from: ndlibersa/usage#36 have already made their way into the system in this commit aead8a8#diff-fd07e4c6441b7582feb727729b3133e2

@jeffnm jeffnm closed this Oct 18, 2016
jeffnm pushed a commit that referenced this pull request Jan 20, 2017
Require required db connection values (like hostname). Fixes #2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants