search

MicrosoftDocs / CloudAppSecurityDocs

Public repo for CloudAppSecurityDocs-pr
Creative Commons Attribution 4.0 International
107 stars 155 forks source link

Fix english typos and grammar in documentation #645

Closed arpitjain099 closed 1 year ago

arpitjain099 commented 1 year ago

Documentation can use some typo fixes and grammar fixes in documentation. This PR fixes some of the issues. Please comment if you'd like me to look at more files

prmerger-automator[bot] commented 1 year ago

@arpitjain099 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

learn-build-service-prod[bot] commented 1 year ago

Learn Build status updates of commit 9767cd0:

:white_check_mark: Validation status: passed

File Status Preview URL Details
.vscode/settings.json :white_check_mark:Succeeded
CloudAppSecurityDocs/aadip-integration.md :white_check_mark:Succeeded
CloudAppSecurityDocs/access-policy-aad.md :white_check_mark:Succeeded
CloudAppSecurityDocs/accounts.md :white_check_mark:Succeeded
CloudAppSecurityDocs/activity-filters.md :white_check_mark:Succeeded
CloudAppSecurityDocs/activity-filters-queries.md :white_check_mark:Succeeded
CloudAppSecurityDocs/activity-privacy.md :white_check_mark:Succeeded
CloudAppSecurityDocs/additional-integrations.md :white_check_mark:Succeeded
CloudAppSecurityDocs/admin-settings.md :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

prmerger-automator[bot] commented 1 year ago

PRMerger Results

Issue Description
Deleted File(s) This PR deletes Markdown or YAML files owned by another author, or json file(s). The pull request must contain a comment from the pull request author confirming that all the file deletions are intentional before the pull request can be merged.
File Change Percent This PR contains file(s) with more than 30% file change.
GDPR Content(s) PR references GDPR and must be human reviewed to ensure appropriate guidelines are followed.
Json File This PR contains json file(s).
learn-build-service-prod[bot] commented 1 year ago

Learn Build status updates of commit d6dd0f3:

:white_check_mark: Validation status: passed

File Status Preview URL Details
.vscode/settings.json :white_check_mark:Succeeded n/a (file deleted or renamed)
CloudAppSecurityDocs/aadip-integration.md :white_check_mark:Succeeded
CloudAppSecurityDocs/access-policy-aad.md :white_check_mark:Succeeded
CloudAppSecurityDocs/accounts.md :white_check_mark:Succeeded
CloudAppSecurityDocs/activity-filters.md :white_check_mark:Succeeded
CloudAppSecurityDocs/activity-filters-queries.md :white_check_mark:Succeeded
CloudAppSecurityDocs/activity-privacy.md :white_check_mark:Succeeded
CloudAppSecurityDocs/additional-integrations.md :white_check_mark:Succeeded
CloudAppSecurityDocs/admin-settings.md :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

prmerger-automator[bot] commented 1 year ago

PRMerger Results

Issue Description
Deleted File(s) This PR deletes Markdown or YAML files owned by another author, or json file(s). The pull request must contain a comment from the pull request author confirming that all the file deletions are intentional before the pull request can be merged.
File Change Percent This PR contains file(s) with more than 30% file change.
GDPR Content(s) PR references GDPR and must be human reviewed to ensure appropriate guidelines are followed.
Json File This PR contains json file(s).
learn-build-service-prod[bot] commented 1 year ago

Learn Build status updates of commit 24c20c3:

:white_check_mark: Validation status: passed

File Status Preview URL Details
.vscode/settings.json :white_check_mark:Succeeded
CloudAppSecurityDocs/aadip-integration.md :white_check_mark:Succeeded
CloudAppSecurityDocs/access-policy-aad.md :white_check_mark:Succeeded
CloudAppSecurityDocs/accounts.md :white_check_mark:Succeeded
CloudAppSecurityDocs/activity-filters.md :white_check_mark:Succeeded
CloudAppSecurityDocs/activity-filters-queries.md :white_check_mark:Succeeded
CloudAppSecurityDocs/activity-privacy.md :white_check_mark:Succeeded
CloudAppSecurityDocs/additional-integrations.md :white_check_mark:Succeeded
CloudAppSecurityDocs/admin-settings.md :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

prmerger-automator[bot] commented 1 year ago

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.
GDPR Content(s) PR references GDPR and must be human reviewed to ensure appropriate guidelines are followed.
Json File This PR contains json file(s).
Court72 commented 1 year ago

@arpitjain099 - It looks like a stray config file made its way into your working branch. Can you remove the .vscode/settings.json file from this PR?

Court72 commented 1 year ago

@batamig, @dcurwin

Can you review the proposed changes?

When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

label:"aq-pr-triaged"

@MicrosoftDocs/public-repo-pr-review-team

learn-build-service-prod[bot] commented 1 year ago

Learn Build status updates of commit 0286279:

:white_check_mark: Validation status: passed

File Status Preview URL Details
.vscode/settings.json :white_check_mark:Succeeded
CloudAppSecurityDocs/aadip-integration.md :white_check_mark:Succeeded
CloudAppSecurityDocs/access-policy-aad.md :white_check_mark:Succeeded
CloudAppSecurityDocs/accounts.md :white_check_mark:Succeeded
CloudAppSecurityDocs/activity-filters.md :white_check_mark:Succeeded
CloudAppSecurityDocs/activity-filters-queries.md :white_check_mark:Succeeded
CloudAppSecurityDocs/activity-privacy.md :white_check_mark:Succeeded
CloudAppSecurityDocs/additional-integrations.md :white_check_mark:Succeeded
CloudAppSecurityDocs/admin-settings.md :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

prmerger-automator[bot] commented 1 year ago

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.
GDPR Content(s) PR references GDPR and must be human reviewed to ensure appropriate guidelines are followed.
Json File This PR contains json file(s).
batamig commented 1 year ago

Hi @arpitjain099 - thanks for your contributions to docs! We've reviewed the PR and made some slight changes and will be merging shortly.