{"html_url": "https://github.com/simonw/datasette/issues/185#issuecomment-376585911", "issue_url": "https://api.github.com/repos/simonw/datasette/issues/185", "id": 376585911, "node_id": "MDEyOklzc3VlQ29tbWVudDM3NjU4NTkxMQ==", "user": {"value": 9599, "label": "simonw"}, "created_at": "2018-03-27T16:19:43Z", "updated_at": "2018-03-27T16:19:43Z", "author_association": "OWNER", "body": "OK, I have an implementation of this. I realised that not ALL metadata should be inherited: it makes sense for source/source_url/license/license_url to be inherited, but it doesn't make sense for the title and description to be inherited down to the individual databases and tables.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "issue": {"value": 299760684, "label": "Metadata should be a nested arbitrary KV store"}, "performed_via_github_app": null}
{"html_url": "https://github.com/simonw/datasette/issues/185#issuecomment-376587017", "issue_url": "https://api.github.com/repos/simonw/datasette/issues/185", "id": 376587017, "node_id": "MDEyOklzc3VlQ29tbWVudDM3NjU4NzAxNw==", "user": {"value": 9599, "label": "simonw"}, "created_at": "2018-03-27T16:22:59Z", "updated_at": "2018-03-27T16:22:59Z", "author_association": "OWNER", "body": "One thing that's missing from this: if you set source/license data at the individual database level they should be inherited by tables within that database.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "issue": {"value": 299760684, "label": "Metadata should be a nested arbitrary KV store"}, "performed_via_github_app": null}
{"html_url": "https://github.com/simonw/datasette/issues/185#issuecomment-376589591", "issue_url": "https://api.github.com/repos/simonw/datasette/issues/185", "id": 376589591, "node_id": "MDEyOklzc3VlQ29tbWVudDM3NjU4OTU5MQ==", "user": {"value": 9599, "label": "simonw"}, "created_at": "2018-03-27T16:30:51Z", "updated_at": "2018-03-27T16:30:51Z", "author_association": "OWNER", "body": "Also needed: the ability to unset metadata. If the root metadata specifies a license_url it should be possible to set \"license_url\": null on a child database or table. The current implementation will ignore null (or empty string) values and default to the top level value.\r\n\r\nI think the templates themselves should be able to indicate if they want the inherited values or not. That way we could support arbitrary key/values and avoid the application code having special knowledge of license_url etc.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "issue": {"value": 299760684, "label": "Metadata should be a nested arbitrary KV store"}, "performed_via_github_app": null}
{"html_url": "https://github.com/simonw/datasette/issues/188#issuecomment-376594727", "issue_url": "https://api.github.com/repos/simonw/datasette/issues/188", "id": 376594727, "node_id": "MDEyOklzc3VlQ29tbWVudDM3NjU5NDcyNw==", "user": {"value": 9599, "label": "simonw"}, "created_at": "2018-03-27T16:46:49Z", "updated_at": "2018-05-28T21:34:34Z", "author_association": "OWNER", "body": "One point of complexity: datasette can be used to bundle multiple .db files into a single \"app\".\r\n\r\nI think that's OK. We could require that the `datasette_files` table is present in the first database file passed on the command-line. Or we could even construct a search path and consult multiple versions of the table spread across multiple files.\r\n\r\nThat said... any configuration that corresponds to a specific table should live in the same database file as that table.\r\n\r\nDitto for general metadata: if we have license/source information for a specific table or database that information should be able to live in the same .db file as the data.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "issue": {"value": 309047460, "label": "Ability to bundle metadata and templates inside the SQLite file"}, "performed_via_github_app": null}
{"html_url": "https://github.com/simonw/datasette/issues/185#issuecomment-376604558", "issue_url": "https://api.github.com/repos/simonw/datasette/issues/185", "id": 376604558, "node_id": "MDEyOklzc3VlQ29tbWVudDM3NjYwNDU1OA==", "user": {"value": 9599, "label": "simonw"}, "created_at": "2018-03-27T17:16:27Z", "updated_at": "2018-03-27T17:16:27Z", "author_association": "OWNER", "body": "I am SO inspired by what you've done with https://salaries.news.baltimoresun.com/ - that's pretty much my ideal use-case for Datasette, and it's by far the most elaborate customization I've seen so far. I'd love to hear other ideas that came up while building that.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "issue": {"value": 299760684, "label": "Metadata should be a nested arbitrary KV store"}, "performed_via_github_app": null}
{"html_url": "https://github.com/simonw/datasette/issues/185#issuecomment-376590265", "issue_url": "https://api.github.com/repos/simonw/datasette/issues/185", "id": 376590265, "node_id": "MDEyOklzc3VlQ29tbWVudDM3NjU5MDI2NQ==", "user": {"value": 222245, "label": "carlmjohnson"}, "created_at": "2018-03-27T16:32:51Z", "updated_at": "2018-03-27T16:32:51Z", "author_association": "NONE", "body": ">I think the templates themselves should be able to indicate if they want the inherited values or not. That way we could support arbitrary key/values and avoid the application code having special knowledge of license_url etc.\r\n\r\nYes, you could have `metadata` that works like `metadata` does currently and `inherited_metadata` that works with inheritance.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "issue": {"value": 299760684, "label": "Metadata should be a nested arbitrary KV store"}, "performed_via_github_app": null}
{"html_url": "https://github.com/simonw/datasette/issues/185#issuecomment-376592044", "issue_url": "https://api.github.com/repos/simonw/datasette/issues/185", "id": 376592044, "node_id": "MDEyOklzc3VlQ29tbWVudDM3NjU5MjA0NA==", "user": {"value": 222245, "label": "carlmjohnson"}, "created_at": "2018-03-27T16:38:23Z", "updated_at": "2018-03-27T16:38:23Z", "author_association": "NONE", "body": "It would be nice to also allow arbitrary keys (maybe under a parent key called params or something to prevent conflicts). For our datasette project, we just have a bunch of dictionaries defined in the base template for things like site URL and column humanized names:  https://github.com/baltimore-sun-data/salaries-datasette/blob/master/templates/base.html It would be cleaner if this were in the metadata.json.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "issue": {"value": 299760684, "label": "Metadata should be a nested arbitrary KV store"}, "performed_via_github_app": null}
{"html_url": "https://github.com/simonw/datasette/issues/185#issuecomment-376614973", "issue_url": "https://api.github.com/repos/simonw/datasette/issues/185", "id": 376614973, "node_id": "MDEyOklzc3VlQ29tbWVudDM3NjYxNDk3Mw==", "user": {"value": 222245, "label": "carlmjohnson"}, "created_at": "2018-03-27T17:49:00Z", "updated_at": "2018-03-27T17:49:00Z", "author_association": "NONE", "body": "@simonw Other than metadata, the biggest item on wishlist for the salaries project was the ability to reorder by column. Of course, that could be done with a custom SQL query, but we didn't want to have to reimplement all the nav/pagination stuff from scratch. \r\n\r\n@carolinp, feel free to add your thoughts.\r\n", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "issue": {"value": 299760684, "label": "Metadata should be a nested arbitrary KV store"}, "performed_via_github_app": null}