Skip to content

Conversation

@shivani-sumo
Copy link
Collaborator

@shivani-sumo shivani-sumo commented Feb 21, 2025

rename app "microsoft defender for cloud" to "azure security defender for cloud"

Purpose of this pull request

This pull request...

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

https://sumologic.atlassian.net/browse/CONN-4195

@cla-bot cla-bot bot added the cla-signed Contributor approved, listed in .clabot file label Feb 21, 2025
Copy link
Collaborator

@JV0812 JV0812 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@kimsauce kimsauce left a comment

Choose a reason for hiding this comment

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

@shivani-sumo @JV0812 I'd also....

  1. Change the page file title from microsoft-defender-for-cloud.md to azure-security-defender-for-cloud.md.
  2. In sidebars.ts, rename microsoft-defender-for-cloud to azure-security-defender-for-cloud and place it in alphabetical order.
  3. Create a 301 redirect in cid-redirects.json.
  4. Update the product list page.
  5. Update any existing links.
  6. Run a global docs search for "Microsoft Defender for Cloud" as it's mentioned in 13 different docs total. You'd probably need to update those, too.

I've set it to draft in the meantime.

@kimsauce kimsauce added the do-not-merge Pull requests that should not be merged label Feb 22, 2025
@kimsauce kimsauce marked this pull request as draft February 22, 2025 01:04
@JV0812
Copy link
Collaborator

JV0812 commented Feb 24, 2025

Thanks for the catch Kim, I have incorporated your comments.
But since name change is not occurred in the app dashboard, the existing name is maintained to all the dashboard content, dashboard image, and S3 folder.

@JV0812 JV0812 requested a review from kimsauce February 24, 2025 06:28
@JV0812 JV0812 marked this pull request as ready for review February 24, 2025 06:28
Co-authored-by: Kim (Sumo Logic) <56411016+kimsauce@users.noreply.github.com>
@ankitgoelcmu ankitgoelcmu added this pull request to the merge queue Feb 25, 2025
Merged via the queue into SumoLogic:main with commit 7f37668 Feb 25, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed Contributor approved, listed in .clabot file do-not-merge Pull requests that should not be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants