Skip to content

Rename Health events to Account events #5582

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JV0812
Copy link
Collaborator

@JV0812 JV0812 commented Jul 11, 2025

Purpose of this pull request

This pull request is to rename the Health events to Account events.

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)

Health Events to Account Events - Name change

@JV0812 JV0812 requested review from kimsauce, jpipkin1, mafsumo, amee-sumo and a team as code owners July 11, 2025 10:53
@cla-bot cla-bot bot added the cla-signed Contributor approved, listed in .clabot file label Jul 11, 2025
@JV0812 JV0812 self-assigned this Jul 11, 2025
@JV0812 JV0812 added do-not-merge Pull requests that should not be merged and removed cla-signed Contributor approved, listed in .clabot file labels Jul 11, 2025
title: Health Events Management APIs
sidebar_label: Health Events
description: Use HTTP endpoints to view your account's health events.
id: account-events
Copy link
Collaborator

Choose a reason for hiding this comment

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

If you're changing the ID of this file from health-events to account-events, you should also change the name of the file from health-events.md to account-events.md.

@@ -3,7 +3,7 @@
Add content to the following sections in this sidebar Navigation file:
Copy link
Collaborator

@jpipkin1 jpipkin1 Jul 11, 2025

Choose a reason for hiding this comment

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

To resolve the build break, in the sidebars.ts file you need to change manage/health-events to manage/account-events, and change api/health-events to api/account-events.

@@ -1,6 +1,6 @@
---
id: health-events
title: Health Events
id: account-events
Copy link
Collaborator

Choose a reason for hiding this comment

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

Just like for that other file, if you're changing the ID of this file from health-events to account-events, you should also change the name of the file from health-events.md to account-events.md.

| KR | https://api.kr.sumologic.com/docs/#tag/healthEvents |
| US1 | https://api.sumologic.com/docs/#tag/healthEvents |
| US2 | https://api.us2.sumologic.com/docs/#tag/healthEvents |
| AU | https://api.au.sumologic.com/docs/#tag/accountEvents |

Choose a reason for hiding this comment

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

This is not going to work since these API docs link are generated from backend and this would require changing the health events API. Is this something that is being planned for future? @SumoLogic/core-platform team owns these APIs and as far as I know this hasn't been communicated to the team that owns these APIs?


Hover your mouse over the status icon to view a tooltip with a count of the detected errors and warnings. You can click on the status icon to open a Health Events panel with details on each detected issue.
Hover your mouse over the status icon to view a tooltip with a count of the detected errors and warnings. You can click on the status icon to open a Account Events panel with details on each detected issue.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Hover your mouse over the status icon to view a tooltip with a count of the detected errors and warnings. You can click on the status icon to open a Account Events panel with details on each detected issue.
Hover your mouse over the status icon to view a tooltip with a count of the detected errors and warnings. You can click on the status icon to open an Account Events panel with details on each detected issue.

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.

You'd also need to add the URL changes to cid-directs.json

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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