search

tyconsulting / azurepolicy

MIT License
147 stars 132 forks source link

Added Storage Account as destination and param for diag settings name #5

Closed ManuInNZ closed 4 years ago

ManuInNZ commented 5 years ago

Hi

Thanks for the base, great work. Just added a param for the Diagnostics setting name and the Storage Account destination.

Cheers Manu

ManuInNZ commented 4 years ago

arf pretty sure it is not as good as you base thing. when I applied it I had a few issues Sorry about that, I forgot about the PR. Might need a wee bit of rework

Cheers Manu

Le ven. 17 avr. 2020 à 20:22, Tao Yang notifications@github.com a écrit :

@tyconsulting approved this pull request.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/tyconsulting/azurepolicy/pull/5#pullrequestreview-395255657, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB4R3ABVU4IMD5PJYHEXDA3RNAGVDANCNFSM4I5WNY7A .