{"action":"closed","issue":{"url":"https://api.github.com/repos/Paymentsense/Dapper.SimpleSave/issues/6","labels_url":"https://api.github.com/repos/Paymentsense/Dapper.SimpleSave/issues/6/labels{/name}","comments_url":"https://api.github.com/repos/Paymentsense/Dapper.SimpleSave/issues/6/comments","events_url":"https://api.github.com/repos/Paymentsense/Dapper.SimpleSave/issues/6/events","html_url":"https://github.com/Paymentsense/Dapper.SimpleSave/issues/6","id":99207305,"number":6,"title":"[ReadOnly] is a confusing name for an attribute","user":{"login":"bartread","id":4139358,"avatar_url":"https://avatars.githubusercontent.com/u/4139358?v=3","gravatar_id":"","url":"https://api.github.com/users/bartread","html_url":"https://github.com/bartread","followers_url":"https://api.github.com/users/bartread/followers","following_url":"https://api.github.com/users/bartread/following{/other_user}","gists_url":"https://api.github.com/users/bartread/gists{/gist_id}","starred_url":"https://api.github.com/users/bartread/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bartread/subscriptions","organizations_url":"https://api.github.com/users/bartread/orgs","repos_url":"https://api.github.com/users/bartread/repos","events_url":"https://api.github.com/users/bartread/events{/privacy}","received_events_url":"https://api.github.com/users/bartread/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/Paymentsense/Dapper.SimpleSave/labels/change","name":"change","color":"5319e7"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-08-05T13:43:41Z","updated_at":"2015-08-05T14:07:30Z","closed_at":"2015-08-05T14:07:30Z","body":"The use of [ReadOnly] to stop Dapper.SimpleSave saving individual properties makes sense functionally but the name becomes confusing when used on properties that are read-write (i.e., have both getters and setters) but that you don't want to persist to the database, as might be the case for convenience properties added to simplify code (or to avoid breaking an API).\r\n\r\nA better name would be something like [NoSave], [DoNotSave], or even [SimpleSaveIgnore]."}}