Remove quickstart-subnet location patch

[jaymiracola]

This is an unnecessary patch that breaks the composition.

[netlify]

✅ Deploy Preview for crossplane ready!

Name	Link
🔨 Latest commit	08c46c7
🔍 Latest deploy log	https://app.netlify.com/sites/crossplane/deploys/66e46db8c4b06a0008660b28
😎 Deploy Preview	https://deploy-preview-812--crossplane.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	1 paths audited Performance: 86 (🟢 up 9 from production) Accessibility: 90 (🔴 down 2 from production) Best Practices: 83 (no change from production) SEO: 93 (no change from production) PWA: 70 (no change from production) View the detailed breakdown and full score reports

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[jbw976]

Change looks good from cross referencing the resource with its entry in the marketplace. Go ahead and copy this content to the content/v1.17 directory also so it shows up in the latest docs. Thanks @jaymiracola! 🙇‍♂️

[jbw976]

ah interesting, it does look like Subnet is the only resource in this composition that doesn't have a spec.forProvider.location field. Nice catch!

https://marketplace.upbound.io/providers/upbound/provider-azure-network/v1.5.0/resources/network.azure.upbound.io/Subnet/v1beta1

[jbw976]

Oh and also please add the DOC sign-off to your commit too!

https://github.com/crossplane/crossplane/tree/main/contributing#certificate-of-origin

[jaymiracola]

@jbw976 Ive got the second commit signed, but maybe the first needs to be as well?

[jbw976]

yep exactly @jaymiracola, from https://github.com/crossplane/docs/pull/812/checks:

Commit sha: 8fa447c, Author: Jay Miracola, Committer: GitHub; The sign-off is missing.

You could amend that first commit also, or just squash them into a single commit with a sign-off. Let me know if you'd like any help with that!

[jaymiracola]

Thanks @jbw976 ! hopefully its all set now 😅