{"action":"created","issue":{"url":"https://api.github.com/repos/openSUSE/osem/issues/1229","repository_url":"https://api.github.com/repos/openSUSE/osem","labels_url":"https://api.github.com/repos/openSUSE/osem/issues/1229/labels{/name}","comments_url":"https://api.github.com/repos/openSUSE/osem/issues/1229/comments","events_url":"https://api.github.com/repos/openSUSE/osem/issues/1229/events","html_url":"https://github.com/openSUSE/osem/pull/1229","id":196093909,"number":1229,"title":"Fixes for PostreSQL support","user":{"login":"commandprompt","id":339156,"avatar_url":"https://avatars.githubusercontent.com/u/339156?v=3","gravatar_id":"","url":"https://api.github.com/users/commandprompt","html_url":"https://github.com/commandprompt","followers_url":"https://api.github.com/users/commandprompt/followers","following_url":"https://api.github.com/users/commandprompt/following{/other_user}","gists_url":"https://api.github.com/users/commandprompt/gists{/gist_id}","starred_url":"https://api.github.com/users/commandprompt/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/commandprompt/subscriptions","organizations_url":"https://api.github.com/users/commandprompt/orgs","repos_url":"https://api.github.com/users/commandprompt/repos","events_url":"https://api.github.com/users/commandprompt/events{/privacy}","received_events_url":"https://api.github.com/users/commandprompt/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-12-16T16:09:28Z","updated_at":"2016-12-19T19:02:42Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/openSUSE/osem/pulls/1229","html_url":"https://github.com/openSUSE/osem/pull/1229","diff_url":"https://github.com/openSUSE/osem/pull/1229.diff","patch_url":"https://github.com/openSUSE/osem/pull/1229.patch"},"body":"Hi, I've made a bunch of fixes in order to make OSEM run on PosgreSQL, I would appreciate if someone can take a look at this.\r\nHere is a brief overview of changes I've done:\r\n\r\n1. fix database migrations to use the correct _boolean_ type of default values for boolean columns (0 -> false) commit c7cc8f8f34090baa1641013af8c3c6aa168052e9\r\n2. contitionally use the add_monetize column instead of add_money for the ticket price DB migration; commit 06d8f6d3685e60042fa06458626e5e69c87d66aa\r\n3. add missing columns to the ActiveRecord DB querie so Postgres does not complain about the missing columns in GROUP By statements; commit 02cdf05b5d77d7878681029e5c9ba491e27f0908\r\n4. add .count(:all) ActiveRecord call to force inclusion of the \"count_all\" column in query result set in queries which referred to this column (this happens automaticall for other DBs, but not for Postgres); commit 02cdf05b5d77d7878681029e5c9ba491e27f0908\r\n5. add UUID monkey patch to solve this ActiveUUID issue https://github.com/jashmenn/activeuuid/issues/74; commit a799ea7ca9cbf53eb6058bf636cf289694d16572\r\n6. add PG database driver gem into the project's Gemfile; commit 78824559136b2c8424a44cae1a9d084ada8b212c\r\n7. add updated schema.rb to the project source tree; commit 8903d88ef9cce38c50443d20008fad76ec65968b\r\n8. fix the column types for ahoy_events.visit_id to use INTEGER instead of BINARY(16), so it matches with the FK type on the visits table (this has nothing to do with Postgres, just a bug I've spotted in the process); commit 5e7d5d60d838e28b4e731568ff43349338849a46\r\n\r\nThanks"},"comment":{"url":"https://api.github.com/repos/openSUSE/osem/issues/comments/268048625","html_url":"https://github.com/openSUSE/osem/pull/1229#issuecomment-268048625","issue_url":"https://api.github.com/repos/openSUSE/osem/issues/1229","id":268048625,"user":{"login":"differentreality","id":4322737,"avatar_url":"https://avatars.githubusercontent.com/u/4322737?v=3","gravatar_id":"","url":"https://api.github.com/users/differentreality","html_url":"https://github.com/differentreality","followers_url":"https://api.github.com/users/differentreality/followers","following_url":"https://api.github.com/users/differentreality/following{/other_user}","gists_url":"https://api.github.com/users/differentreality/gists{/gist_id}","starred_url":"https://api.github.com/users/differentreality/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/differentreality/subscriptions","organizations_url":"https://api.github.com/users/differentreality/orgs","repos_url":"https://api.github.com/users/differentreality/repos","events_url":"https://api.github.com/users/differentreality/events{/privacy}","received_events_url":"https://api.github.com/users/differentreality/received_events","type":"User","site_admin":false},"created_at":"2016-12-19T19:02:42Z","updated_at":"2016-12-19T19:02:42Z","body":"Hey :-) We hope to work more intensively on the open PRs in the coming days! It might take us a bit longer to review larger PRs, so please bear with us! Regarding Travis, it seems it is complaining about some styling thingies and also some other things which I am not sure your PR affects. We will look more closely. \r\nThank you very much for making the effort to make the necessary changes for postgresql and taking the time to send a PR for it!"}}
{"actor":{"display_login":"differentreality"}}