{"action":"created","issue":{"url":"https://api.github.com/repos/dpla/heidrun-mappings/issues/28","labels_url":"https://api.github.com/repos/dpla/heidrun-mappings/issues/28/labels{/name}","comments_url":"https://api.github.com/repos/dpla/heidrun-mappings/issues/28/comments","events_url":"https://api.github.com/repos/dpla/heidrun-mappings/issues/28/events","html_url":"https://github.com/dpla/heidrun-mappings/pull/28","id":102165058,"number":28,"title":"Add Florida mapping","user":{"login":"markbreedlove","id":5550828,"avatar_url":"https://avatars.githubusercontent.com/u/5550828?v=3","gravatar_id":"","url":"https://api.github.com/users/markbreedlove","html_url":"https://github.com/markbreedlove","followers_url":"https://api.github.com/users/markbreedlove/followers","following_url":"https://api.github.com/users/markbreedlove/following{/other_user}","gists_url":"https://api.github.com/users/markbreedlove/gists{/gist_id}","starred_url":"https://api.github.com/users/markbreedlove/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/markbreedlove/subscriptions","organizations_url":"https://api.github.com/users/markbreedlove/orgs","repos_url":"https://api.github.com/users/markbreedlove/repos","events_url":"https://api.github.com/users/markbreedlove/events{/privacy}","received_events_url":"https://api.github.com/users/markbreedlove/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-08-20T15:26:12Z","updated_at":"2015-08-25T19:48:34Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/dpla/heidrun-mappings/pulls/28","html_url":"https://github.com/dpla/heidrun-mappings/pull/28","diff_url":"https://github.com/dpla/heidrun-mappings/pull/28.diff","patch_url":"https://github.com/dpla/heidrun-mappings/pull/28.patch"},"body":"I believe that this should take care of the Florida mapping, pending QA; or it's pretty close.  I've done a little QA myself locally, on 1,000 records in my Krikri QA app, and it looks like the results are not completely crazy.\r\n\r\nThere were some earlier pull requests:\r\n* Original pull request, just for discussion: https://github.com/dpla/heidrun-mappings/pull/21\r\n* Feedback on genre changes: https://github.com/dpla/heidrun-mappings/pull/26\r\n\r\nI'm going to close those now and this one can take up the conversation.\r\n\r\nI believe that the issues raised in the original one have been addressed, including the absence of the dcformat mapping.\r\n\r\nThe most significant addition here, I think, is the `MappingTools` module, which should make it much easier to map our two other MARC providers in the future, and which has made the present mapping easier to read and less repetitive than it would have been otherwise."},"comment":{"url":"https://api.github.com/repos/dpla/heidrun-mappings/issues/comments/134717318","html_url":"https://github.com/dpla/heidrun-mappings/pull/28#issuecomment-134717318","issue_url":"https://api.github.com/repos/dpla/heidrun-mappings/issues/28","id":134717318,"user":{"login":"markbreedlove","id":5550828,"avatar_url":"https://avatars.githubusercontent.com/u/5550828?v=3","gravatar_id":"","url":"https://api.github.com/users/markbreedlove","html_url":"https://github.com/markbreedlove","followers_url":"https://api.github.com/users/markbreedlove/followers","following_url":"https://api.github.com/users/markbreedlove/following{/other_user}","gists_url":"https://api.github.com/users/markbreedlove/gists{/gist_id}","starred_url":"https://api.github.com/users/markbreedlove/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/markbreedlove/subscriptions","organizations_url":"https://api.github.com/users/markbreedlove/orgs","repos_url":"https://api.github.com/users/markbreedlove/repos","events_url":"https://api.github.com/users/markbreedlove/events{/privacy}","received_events_url":"https://api.github.com/users/markbreedlove/received_events","type":"User","site_admin":false},"created_at":"2015-08-25T19:48:34Z","updated_at":"2015-08-25T19:48:34Z","body":"`MappingTools` has been moved out of this project, and added to `heidrun`: https://github.com/dpla/heidrun/pull/32"}}