Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update sample5.vb #956

Merged
merged 5 commits into from
Apr 11, 2024
Merged

Update sample5.vb #956

merged 5 commits into from
Apr 11, 2024

Conversation

6eh01der
Copy link
Contributor

Correct VBS code (current is not VBS, but JavaScript).

Correct VBS code (current is JavaScript).
@6eh01der 6eh01der requested a review from a team as a code owner January 12, 2023 20:47
@6eh01der
Copy link
Contributor Author

@microsoft-github-policy-service agree

Copy link
Contributor

@wadepickett wadepickett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving with a small fix.

iis/configuration/moduleProviders/index/samples/sample5.vb Outdated Show resolved Hide resolved
Correcting back `moduleProvidersCollection`
@wadepickett
Copy link
Contributor

wadepickett commented Apr 11, 2024

Thanks @6eh01der! Greatly appreciated you spotted this issue and provided a VBScript fix!

@wadepickett wadepickett reopened this Apr 11, 2024
@wadepickett
Copy link
Contributor

Forced refresh build close/re-open

@wadepickett
Copy link
Contributor

Just realized I added an error, my apologies, fixing...

Fixed my error, ensured `modulesSection` is used throughout.
@wadepickett
Copy link
Contributor

I needed to change the filename to a .vbs extension, it was left as .vb and the doc that refers to it therefore needs a link update.

@wadepickett
Copy link
Contributor

wadepickett commented Apr 11, 2024

Verified in Internal review doc build

@wadepickett wadepickett merged commit 1ed1589 into MicrosoftDocs:main Apr 11, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants