{"action":"created","issue":{"url":"https://api.github.com/repos/code-dot-org/code-dot-org/issues/1639","labels_url":"https://api.github.com/repos/code-dot-org/code-dot-org/issues/1639/labels{/name}","comments_url":"https://api.github.com/repos/code-dot-org/code-dot-org/issues/1639/comments","events_url":"https://api.github.com/repos/code-dot-org/code-dot-org/issues/1639/events","html_url":"https://github.com/code-dot-org/code-dot-org/pull/1639","id":62296281,"number":1639,"title":"Add 'list tables' support for dynamo db, needed for applab data browser.","user":{"login":"davidsbailey","id":8001765,"avatar_url":"https://avatars.githubusercontent.com/u/8001765?v=3","gravatar_id":"","url":"https://api.github.com/users/davidsbailey","html_url":"https://github.com/davidsbailey","followers_url":"https://api.github.com/users/davidsbailey/followers","following_url":"https://api.github.com/users/davidsbailey/following{/other_user}","gists_url":"https://api.github.com/users/davidsbailey/gists{/gist_id}","starred_url":"https://api.github.com/users/davidsbailey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/davidsbailey/subscriptions","organizations_url":"https://api.github.com/users/davidsbailey/orgs","repos_url":"https://api.github.com/users/davidsbailey/repos","events_url":"https://api.github.com/users/davidsbailey/events{/privacy}","received_events_url":"https://api.github.com/users/davidsbailey/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-03-17T05:59:48Z","updated_at":"2015-03-19T18:21:32Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/code-dot-org/code-dot-org/pulls/1639","html_url":"https://github.com/code-dot-org/code-dot-org/pull/1639","diff_url":"https://github.com/code-dot-org/code-dot-org/pull/1639.diff","patch_url":"https://github.com/code-dot-org/code-dot-org/pull/1639.patch"},"body":"Note that I added a new index on the shared_tables table in DynamoDB to make this change work:\r\n\r\nindex: channel_id-table_name-index\r\nhash key: channel_id (Number)\r\nrange key: table_name (String)\r\n\r\nHere is the feature which this supports: http://staging.code.org/private/edit-csp-app/DvTw9X3pDcyDyil44S6qbw==\r\n\r\n![screen shot 2015-03-16 at 11 01 48 pm](https://cloud.githubusercontent.com/assets/8001765/6682126/79ec8fae-cc30-11e4-95a4-e6b3f8bc44c5.png)\r\n"},"comment":{"url":"https://api.github.com/repos/code-dot-org/code-dot-org/issues/comments/83700003","html_url":"https://github.com/code-dot-org/code-dot-org/pull/1639#issuecomment-83700003","issue_url":"https://api.github.com/repos/code-dot-org/code-dot-org/issues/1639","id":83700003,"user":{"login":"wjordan","id":56541,"avatar_url":"https://avatars.githubusercontent.com/u/56541?v=3","gravatar_id":"","url":"https://api.github.com/users/wjordan","html_url":"https://github.com/wjordan","followers_url":"https://api.github.com/users/wjordan/followers","following_url":"https://api.github.com/users/wjordan/following{/other_user}","gists_url":"https://api.github.com/users/wjordan/gists{/gist_id}","starred_url":"https://api.github.com/users/wjordan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wjordan/subscriptions","organizations_url":"https://api.github.com/users/wjordan/orgs","repos_url":"https://api.github.com/users/wjordan/repos","events_url":"https://api.github.com/users/wjordan/events{/privacy}","received_events_url":"https://api.github.com/users/wjordan/received_events","type":"User","site_admin":false},"created_at":"2015-03-19T18:21:32Z","updated_at":"2015-03-19T18:21:32Z","body":"I'm not saying that tests must be written at this point either, it might in fact be too early since this is not yet a 'live' feature and we're still experimenting a bit. Just emphasizing that because I can't manually test that the DynamoDB implementation actually works as part of my code review, manual testing needs to be a regular part of this work's development until the automated tests arrive."}}