home / github / issue_comments

Menu
  • Search all tables
  • GraphQL API

issue_comments: 592999503

This data as json

html_url issue_url id node_id user created_at updated_at author_association body reactions issue performed_via_github_app
https://github.com/simonw/sqlite-utils/issues/46#issuecomment-592999503 https://api.github.com/repos/simonw/sqlite-utils/issues/46 592999503 MDEyOklzc3VlQ29tbWVudDU5Mjk5OTUwMw== 35075 2020-02-29T22:08:20Z 2020-02-29T22:08:20Z NONE

@simonw any thoughts on allow extracts to specify the lookup column name? If I'm understanding the documentation right, .lookup() allows you to define the "value" column (the documentation uses name), but when you use extracts keyword as part of .insert(), .upsert() etc. the lookup must be done against a column named "value". I have an existing lookup table that I've populated with columns "id" and "name" as opposed to "id" and "value", and seems I can't use extracts=, unless I'm missing something...

Initial thought on how to do this would be to allow the dictionary value to be a tuple of table name column pair... so: table = db.table("trees", extracts={"species_id": ("Species", "name"})

I haven't dug too much into the existing code yet, but does this make sense? Worth doing?

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
471780443  
Powered by Datasette · Queries took 0.903ms · About: github-to-sqlite