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

[FIX] structure of sqlite #67

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kurkop
Copy link

@kurkop kurkop commented Jun 2, 2017

Thanks to @nebalceroc

- Use String type for 'place' column in sqlite
fix phutchins#38
@thejeff77
Copy link

I could use this fix. Running into the same issue. Can you assign this pull request to one of the contributors to get it moving?

@melgart
Copy link

melgart commented Sep 3, 2021

I tested the feasibility of this fix by manually changing my sqlite3 table column to a String (from Int), but I still encountered the same problem. ☹️ I would probably next look at the code that parses the value from the 'placeholder' table to see if there's anything there that's attempting to treat the value as an Integer. Thanks for putting up this PR, at least -- helpful.

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.

Exception: BSON::ObjectId::Invalid: 'Infinity' is an invalid ObjectId
3 participants