Skip to content

Conversation

@BWMerlin
Copy link
Contributor

Added the location to where the export zone file can be found (C:\Windows\System32\DNS) and that the location is on the DNS server. Also added a note that -FileName is a filename and not a filepath. This should clarify how users can use this command and where they can locate the output.

Added the location to where the export zone file can be found (C:\Windows\System32\DNS) and that the location is on the DNS server.  Also added a note that -FileName is a filename and not a filepath.  This should clarify how users can use this command and where they can locate the output.
@opbld32
Copy link

opbld32 commented Jun 14, 2021

Docs Build status updates of commit bf90148:

✅ Validation status: passed

File Status Preview URL Details
docset/winserver2019-ps/dnsserver/Export-DnsServerZone.md ✅Succeeded View (WindowsServer2019-ps)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

Copy link
Contributor

@JohanFreelancer9 JohanFreelancer9 left a comment

Choose a reason for hiding this comment

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

Copy review - @BWMerlin, please commit my minor suggested changes. Many thanks.

Co-authored-by: JohanFreelancer9 <48568725+JohanFreelancer9@users.noreply.github.com>
@opbld30
Copy link

opbld30 commented Jun 15, 2021

Docs Build status updates of commit 6086632:

✅ Validation status: passed

File Status Preview URL Details
docset/winserver2019-ps/dnsserver/Export-DnsServerZone.md ✅Succeeded View (WindowsServer2019-ps)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@o0nj o0nj requested review from a user, dariomws and get-itips June 28, 2021 11:10
Copy link
Contributor

@dariomws dariomws left a comment

Choose a reason for hiding this comment

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

I think the same update must be applied to all Windows versions (from 2012 to 2022).

Copy link
Contributor

@get-itips get-itips left a comment

Choose a reason for hiding this comment

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

Tested this, ok, can you please also correct the description for FileName parameter? Thank you!

Removed wording about being able to add a file path when using -FileName as it is NOT possible to use a file path.
@BWMerlin
Copy link
Contributor Author

Tested this, ok, can you please also correct the description for FileName parameter? Thank you!

I think I have made the requested change to -FileName. Please review.

@opbld32
Copy link

opbld32 commented Jul 18, 2021

Docs Build status updates of commit bec3639:

✅ Validation status: passed

File Status Preview URL Details
docset/winserver2019-ps/dnsserver/Export-DnsServerZone.md ✅Succeeded View (WindowsServer2019-ps)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@o0nj o0nj added the Sign off The pull request is ready to be reviewed and merged by PubOps label Jul 19, 2021
@opbld30
Copy link

opbld30 commented Jul 19, 2021

Docs Build status updates of commit 2c42ae9:

✅ Validation status: passed

File Status Preview URL Details
docset/winserver2019-ps/dnsserver/Export-DnsServerZone.md ✅Succeeded View (WindowsServer2019-ps)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@v-dihans v-dihans self-assigned this Jul 19, 2021
@v-dihans v-dihans added the In review PubOps is reviewing the pull request label Jul 19, 2021
@v-dihans v-dihans merged commit acf57a6 into MicrosoftDocs:master Jul 19, 2021
@BWMerlin BWMerlin deleted the patch-1 branch July 22, 2021 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

In review PubOps is reviewing the pull request Sign off The pull request is ready to be reviewed and merged by PubOps

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants