Optimised infrastructure deployment, fixed AppInsights telemetry

[Gordonby]

Purpose

I didn't find that App Insights was correctly wired up, so what started as a simple fix ended up as

• Rewritten Arm JSON as Bicep.
• Optimised the process for getting connectionstring for cosmos, it's all done in the bicep now - no powershell needed
• Optimised the deployment of code to app service, it's all done in the bicep now (pulling from GitHub) - no local az or git commands
• Optimised a bunch of little things in the Arm Json, (param metadata/defaults/etc), bumping up resource API version where I could
• Enabled in app diagnostic logging for App Service
• Fixed the App Service -> App Insights link so telemetry flows
• Simplified the deployment script

Does this introduce a breaking change?

[ ] Yes
[X ] No

Pull Request Type

What kind of change does this Pull Request introduce?

[x] Bugfix
[x] Feature
[x] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[x] Documentation content changes
[ ] Other... Please describe:

How to Test

Follow the readme instructions

[Gordonby]

@issacnitin - I see you bugfixed the deploymentscript a few hours ago. Happy to chat about the PR if you need.

[Gordonby]

Closing due to lack of engagement from repo owners.