{"action":"opened","issue":{"url":"https://api.github.com/repos/natemcmaster/CommandLineUtils/issues/194","repository_url":"https://api.github.com/repos/natemcmaster/CommandLineUtils","labels_url":"https://api.github.com/repos/natemcmaster/CommandLineUtils/issues/194/labels{/name}","comments_url":"https://api.github.com/repos/natemcmaster/CommandLineUtils/issues/194/comments","events_url":"https://api.github.com/repos/natemcmaster/CommandLineUtils/issues/194/events","html_url":"https://github.com/natemcmaster/CommandLineUtils/issues/194","id":391438404,"node_id":"MDU6SXNzdWUzOTE0Mzg0MDQ=","number":194,"title":"Feedback from a first time user of this tool (sen. dev)","user":{"login":"WernerMairl","id":10084630,"node_id":"MDQ6VXNlcjEwMDg0NjMw","avatar_url":"https://avatars1.githubusercontent.com/u/10084630?v=4","gravatar_id":"","url":"https://api.github.com/users/WernerMairl","html_url":"https://github.com/WernerMairl","followers_url":"https://api.github.com/users/WernerMairl/followers","following_url":"https://api.github.com/users/WernerMairl/following{/other_user}","gists_url":"https://api.github.com/users/WernerMairl/gists{/gist_id}","starred_url":"https://api.github.com/users/WernerMairl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/WernerMairl/subscriptions","organizations_url":"https://api.github.com/users/WernerMairl/orgs","repos_url":"https://api.github.com/users/WernerMairl/repos","events_url":"https://api.github.com/users/WernerMairl/events{/privacy}","received_events_url":"https://api.github.com/users/WernerMairl/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-12-16T06:01:00Z","updated_at":"2018-12-16T06:01:00Z","closed_at":null,"author_association":"NONE","body":"Hi\r\ni'm using your tool (and i like it!) the first time, so i have some suggestions about usability.....\r\n\r\n## Finding 1\r\n```\r\nCommandLineApplication.Execute<XXX>()\r\n```\r\n\r\nseems to execute a Method \"OnExecute\" on class XXX by Name (reflection !?)\r\nThe only hint that i found about this was inside the intellisense tooltip (=> increase visibility of how the feature works)\r\nfor me this is a core concept, that can be better presented in the documentation (method must be public or private ? Naming Convention etc...)\r\n\r\nusing the following rows \r\n```\r\napp = new CommandLineApplication<XXX>()\r\n// app.OnExecute ( () => ) explicitely NOT called/implemented\r\napp.execute(args);\r\n```\r\n\r\nthe \"OnExecute\" Method on class XXX seems to not be called.\r\nBug or Intended ?\r\nin every case... improvement of documentation/visibility suggested\r\n\r\n## Finding 2:\r\n\r\nusing the following way, it was \"hard\" to find the Instance of XXX\r\n\r\n```\r\napp = new CommandLineApplication<XXX>()\r\n```\r\n\r\nyes, i have found it as\r\n\r\n```\r\n XXX x = app.Model;\r\n```\r\nusing \"Ctrl+J\" in VS you must scroll down a lot and \"Model\" is not the first wording that me comes in mind ... my first try was something that starts with \"A\" like Application... but there is nothing\r\n\r\nsuggestions: think about rename Property \"Model\" into \"Application\" or something else that helps...\r\n\r\n\r\n## Bottom Line:\r\nonly a few suggestions about usabilty from someone new on this API.\r\nFeel free to close this issue/create single one for my suggestions etc....\r\n\r\nregards\r\nWerner\r\n\r\n\r\n \r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n"}}
{"actor":{"display_login":"WernerMairl"}}