{"action":"created","issue":{"url":"https://api.github.com/repos/MarcosMeli/FileHelpers/issues/174","labels_url":"https://api.github.com/repos/MarcosMeli/FileHelpers/issues/174/labels{/name}","comments_url":"https://api.github.com/repos/MarcosMeli/FileHelpers/issues/174/comments","events_url":"https://api.github.com/repos/MarcosMeli/FileHelpers/issues/174/events","html_url":"https://github.com/MarcosMeli/FileHelpers/pull/174","id":116150257,"number":174,"title":"Implementation of include headers column name on DataTableToCsv","user":{"login":"denistorresan","id":1846826,"avatar_url":"https://avatars.githubusercontent.com/u/1846826?v=3","gravatar_id":"","url":"https://api.github.com/users/denistorresan","html_url":"https://github.com/denistorresan","followers_url":"https://api.github.com/users/denistorresan/followers","following_url":"https://api.github.com/users/denistorresan/following{/other_user}","gists_url":"https://api.github.com/users/denistorresan/gists{/gist_id}","starred_url":"https://api.github.com/users/denistorresan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/denistorresan/subscriptions","organizations_url":"https://api.github.com/users/denistorresan/orgs","repos_url":"https://api.github.com/users/denistorresan/repos","events_url":"https://api.github.com/users/denistorresan/events{/privacy}","received_events_url":"https://api.github.com/users/denistorresan/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-11-10T16:56:18Z","updated_at":"2015-11-12T22:18:47Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/MarcosMeli/FileHelpers/pulls/174","html_url":"https://github.com/MarcosMeli/FileHelpers/pull/174","diff_url":"https://github.com/MarcosMeli/FileHelpers/pull/174.diff","patch_url":"https://github.com/MarcosMeli/FileHelpers/pull/174.patch"},"body":"Hello,\r\nI implemented the ability to save CSV from DataTable with columns name as the first header row.\r\n\r\nHope this featues can be included into dev.\r\n\r\nI added one method:\r\n\r\nCsvEngine.DataTableToCsv(DataTable dt, string filename, bool includeHeaders, CsvOptions options);\r\n\r\nif includeHeaders==true, than the DataTable column name is included.\r\nAlso there's the previous version of that method without the new parameter:\r\n\r\nCsvEngine.DataTableToCsv(DataTable dt, string filename, CsvOptions options);\r\n\r\nthank you!"},"comment":{"url":"https://api.github.com/repos/MarcosMeli/FileHelpers/issues/comments/156253911","html_url":"https://github.com/MarcosMeli/FileHelpers/pull/174#issuecomment-156253911","issue_url":"https://api.github.com/repos/MarcosMeli/FileHelpers/issues/174","id":156253911,"user":{"login":"denistorresan","id":1846826,"avatar_url":"https://avatars.githubusercontent.com/u/1846826?v=3","gravatar_id":"","url":"https://api.github.com/users/denistorresan","html_url":"https://github.com/denistorresan","followers_url":"https://api.github.com/users/denistorresan/followers","following_url":"https://api.github.com/users/denistorresan/following{/other_user}","gists_url":"https://api.github.com/users/denistorresan/gists{/gist_id}","starred_url":"https://api.github.com/users/denistorresan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/denistorresan/subscriptions","organizations_url":"https://api.github.com/users/denistorresan/orgs","repos_url":"https://api.github.com/users/denistorresan/repos","events_url":"https://api.github.com/users/denistorresan/events{/privacy}","received_events_url":"https://api.github.com/users/denistorresan/received_events","type":"User","site_admin":false},"created_at":"2015-11-12T22:18:47Z","updated_at":"2015-11-12T22:18:47Z","body":"Ok so, here my proposal:\r\n\r\n1) on CsvOptions, I'll add property ExportHeaderColumns (or you prefer IncludeHeaderColumns ?), like:\r\n\r\noptions.ExportHeaderColumns = true;\r\n\r\n2) Delete my DataTableToCsv method, and include to the previous one my code\r\n\r\nLet me know if it's ok for you I'll do the changes tomorrow!"}}