{"action":"created","issue":{"url":"https://api.github.com/repos/willthames/photodeck.lrdevplugin/issues/2","labels_url":"https://api.github.com/repos/willthames/photodeck.lrdevplugin/issues/2/labels{/name}","comments_url":"https://api.github.com/repos/willthames/photodeck.lrdevplugin/issues/2/comments","events_url":"https://api.github.com/repos/willthames/photodeck.lrdevplugin/issues/2/events","html_url":"https://github.com/willthames/photodeck.lrdevplugin/issues/2","id":54050060,"number":2,"title":"Some small fixes/improvments","user":{"login":"cmaion","id":187317,"avatar_url":"https://avatars.githubusercontent.com/u/187317?v=3","gravatar_id":"","url":"https://api.github.com/users/cmaion","html_url":"https://github.com/cmaion","followers_url":"https://api.github.com/users/cmaion/followers","following_url":"https://api.github.com/users/cmaion/following{/other_user}","gists_url":"https://api.github.com/users/cmaion/gists{/gist_id}","starred_url":"https://api.github.com/users/cmaion/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cmaion/subscriptions","organizations_url":"https://api.github.com/users/cmaion/orgs","repos_url":"https://api.github.com/users/cmaion/repos","events_url":"https://api.github.com/users/cmaion/events{/privacy}","received_events_url":"https://api.github.com/users/cmaion/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-01-12T12:51:23Z","updated_at":"2015-01-13T08:05:50Z","closed_at":"2015-01-13T08:05:50Z","body":"Hi Will,\r\n\r\nThanks for this plugin and sharing it!\r\n\r\nI had a quick look this morning and did some updates to your code while doing so:\r\n\r\n1. couple of minor fixes\r\n2. cache some API replies for the duration of the LR session (websites & gallery display styles)\r\n3. don't replace photos with identical file names at upload (this is dangerous for users that don't care about file name uniqueness)\r\n4. start reducing the use of the \"full gallery listing\" API endpoint, as it can be a bit slow/taxing on huge sites. More needs to be done here ;)\r\n5. fix root gallery discovery when root gallery is not named \"Galleries\"\r\n\r\nMy fork is here https://github.com/cmaion/photodeck.lrdevplugin.\r\nFeel free to cherry pick/merge!\r\n\r\nThanks,\r\nCedric"},"comment":{"url":"https://api.github.com/repos/willthames/photodeck.lrdevplugin/issues/comments/69708561","html_url":"https://github.com/willthames/photodeck.lrdevplugin/issues/2#issuecomment-69708561","issue_url":"https://api.github.com/repos/willthames/photodeck.lrdevplugin/issues/2","id":69708561,"user":{"login":"willthames","id":391807,"avatar_url":"https://avatars.githubusercontent.com/u/391807?v=3","gravatar_id":"","url":"https://api.github.com/users/willthames","html_url":"https://github.com/willthames","followers_url":"https://api.github.com/users/willthames/followers","following_url":"https://api.github.com/users/willthames/following{/other_user}","gists_url":"https://api.github.com/users/willthames/gists{/gist_id}","starred_url":"https://api.github.com/users/willthames/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/willthames/subscriptions","organizations_url":"https://api.github.com/users/willthames/orgs","repos_url":"https://api.github.com/users/willthames/repos","events_url":"https://api.github.com/users/willthames/events{/privacy}","received_events_url":"https://api.github.com/users/willthames/received_events","type":"User","site_admin":false},"created_at":"2015-01-13T08:05:50Z","updated_at":"2015-01-13T08:05:50Z","body":"I'll close this issue and will add further comments to #3. Thanks for the PR"}}