NOISSUE-0: added -user parameter to Send-GraphMailMessage #13
Conversation
…cumentation, added parameterhelp, tested with a few users, including me (which requires delegated permissions instead of app permissions)
|
@jhoneill what do you think about my changes? What can I improve so you will merge it into your repository? KR Fabian |
|
@jhoneill please reply |
|
@fasteiner Sorry this project has been frozen for a few months because of other things happening and trying to do it alone. It causes issues for maintainers if someone wants to reformat the code, move the help around etc. because in a sea of changes there are are handful of lines which have changed, and one function in among many now conforms to a different set of conventions layout conventions. Even if some of practices look "wrong", if they aren't affecting the result "fixing" them with another change just complicates things. But: your idea is a good one and I'll look to merge it in. I'm now working on the project again - a client needs the functionality and would benefit from a few additions - so this should happen soon. |
2 participants
added -user parameter to Send-GraphMailMessage, updated documentation, added parameterhelp, tested with a few users, including me (which requires delegated permissions instead of app permissions)