{"html_url": "https://github.com/simonw/datasette/issues/1533#issuecomment-981149039", "issue_url": "https://api.github.com/repos/simonw/datasette/issues/1533", "id": 981149039, "node_id": "IC_kwDOBm6k_c46eyVv", "user": {"value": 9599, "label": "simonw"}, "created_at": "2021-11-28T20:45:36Z", "updated_at": "2021-11-28T20:45:36Z", "author_association": "OWNER", "body": "I built an initial prototype of this in a branch: https://github.com/simonw/datasette/commit/e0a84691c2959f2d1d76948574c9c4a910c7556c - which exposed even more flaws in the way `TableView` is structured (adding custom HTTP headers to the response is way harder than it should be) which I should address in the refactor in #617.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "issue": {"value": 1065431383, "label": "Add `Link: rel=\"alternate\"` header pointing to JSON for a table/query"}, "performed_via_github_app": null}
{"html_url": "https://github.com/simonw/datasette/issues/1534#issuecomment-981149531", "issue_url": "https://api.github.com/repos/simonw/datasette/issues/1534", "id": 981149531, "node_id": "IC_kwDOBm6k_c46eydb", "user": {"value": 9599, "label": "simonw"}, "created_at": "2021-11-28T20:48:54Z", "updated_at": "2021-11-28T20:48:54Z", "author_association": "OWNER", "body": "If I'm going to do this, is there value in also spotting `Accept: text/csv` and returning CSV for that? I'm pretty sure no client has EVER implemented this though, so it feels like it would be showboating.", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "issue": {"value": 1065432388, "label": "Maybe return JSON from HTML pages if `Accept: application/json` is sent"}, "performed_via_github_app": null}
{"html_url": "https://github.com/simonw/datasette/issues/1518#issuecomment-981153186", "issue_url": "https://api.github.com/repos/simonw/datasette/issues/1518", "id": 981153186, "node_id": "IC_kwDOBm6k_c46ezWi", "user": {"value": 9599, "label": "simonw"}, "created_at": "2021-11-28T21:13:50Z", "updated_at": "2021-11-28T21:13:50Z", "author_association": "OWNER", "body": "I'm also going to use the new `datasette-table` Web Component to help guide the design of the new API, which relates directly to this issue too:\r\n\r\n- #1532", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "issue": {"value": 1058072543, "label": "Complete refactor of TableView and table.html template"}, "performed_via_github_app": null}
{"html_url": "https://github.com/simonw/datasette/issues/1518#issuecomment-981172385", "issue_url": "https://api.github.com/repos/simonw/datasette/issues/1518", "id": 981172385, "node_id": "IC_kwDOBm6k_c46e4Ch", "user": {"value": 9599, "label": "simonw"}, "created_at": "2021-11-28T23:21:26Z", "updated_at": "2021-11-28T23:21:26Z", "author_association": "OWNER", "body": "Aside: is there any reason this work can't complete the long-running goal of merging the TableView and QueryView, such that most of the features available for tables become available for arbitrary queries too?\r\n\r\nI had already mentally committed to implementing facets for queries, but I just realized that filters could work too - using either a CTE or a nested query.\r\n\r\nPagination is the one holdout here, since table pagination uses keyset pagination over a known order. But maybe arbitrary queries can only be paginated off you order them first?", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "issue": {"value": 1058072543, "label": "Complete refactor of TableView and table.html template"}, "performed_via_github_app": null}
{"html_url": "https://github.com/simonw/datasette/issues/1518#issuecomment-981172801", "issue_url": "https://api.github.com/repos/simonw/datasette/issues/1518", "id": 981172801, "node_id": "IC_kwDOBm6k_c46e4JB", "user": {"value": 9599, "label": "simonw"}, "created_at": "2021-11-28T23:23:51Z", "updated_at": "2021-11-28T23:23:51Z", "author_association": "OWNER", "body": "(I could experiment with merging the two tables by adding a temporary undocumented `?_sql=` parameter to the in-progress table view that sets an alternative query instead of `select cols from table` - added bonus, this will force me to use introspection against the returned columns rather than mixing in the known columns for the specified table)", "reactions": "{\"total_count\": 0, \"+1\": 0, \"-1\": 0, \"laugh\": 0, \"hooray\": 0, \"confused\": 0, \"heart\": 0, \"rocket\": 0, \"eyes\": 0}", "issue": {"value": 1058072543, "label": "Complete refactor of TableView and table.html template"}, "performed_via_github_app": null}