-
Notifications
You must be signed in to change notification settings - Fork 21
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
Cannot view /feeds after updating to latest code. #71
Comments
Are you using the code from the 0.9.3 release (which is current master codebase)? It looks like the /feeds query has been improperly written. Also, which version of Pewee and database backend are you using? |
Yes, on master:
Installed packages:
I am using PostgreSQL:
HTH! |
Cannot reproduce the problem with a fresh installation cloned from master. I tried both with SQLite and MySQL. Unfortunately right now I cannot test with PostgreSQL. I was running Peewee 2.2.3, but even after the upgrade to version 2.2.5 things ran smoothly. Can you reproduce the error with SQLite? |
Just tried to reproduce in sqlite, and could not. It appears to be a Peewee issue with PostgreSQL. |
Ha! I think I figured out what the problem is. I'll fix & test more thoroughly in the next release. However, in the meantime if you want to patch by yourself open up coldsweat/frontend.py file and at line 555 replace the def get_feeds(user, *select):
select = select or [Feed, Icon, fn.Count(Entry.id).alias('entries')]
q = Feed.select(*select).join(Icon).switch(Feed).join(Entry, JOIN_LEFT_OUTER).switch(Feed).join(Subscription).where(Subscription.user == user).group_by(Feed, Icon)
return q |
That indeed did work. Its a bit slow, but at least its not crashing now :) |
The good news is that Icon model will be removed in the future (see #26) so we'll spare that join/group_by too. |
I get the following stack trace:
The text was updated successfully, but these errors were encountered: