-
Notifications
You must be signed in to change notification settings - Fork 409
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 firewall-requirements.md #3557
Conversation
line 116 - resolve goes from azure local to AD. not the other way around.
@YannickDeKoeijer : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
Learn Build status updates of commit 9d532ec: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
@YannickDeKoeijer can you remove the "fv" added at the end of line 101 please? It breaks the table. Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
fixed accidental extra FV text
@ttorble changed, (removed the FV text) must have been a mistake. |
Learn Build status updates of commit 3623f39: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
#sign-off |
line 116 - resolve goes from azure local to AD. not the other way around.