Skip to content

Add Classic Central Alert Inventory Workflow - #156

Open
KarthikSKumar98 wants to merge 4 commits into
v2from
transition-alert-inventory
Open

Add Classic Central Alert Inventory Workflow#156
KarthikSKumar98 wants to merge 4 commits into
v2from
transition-alert-inventory

Conversation

@KarthikSKumar98

Copy link
Copy Markdown
Collaborator

This PR adds a read-only workflow that retrieves enabled alert configurations from Classic Central. It provides scope-aware browser review and exports results as JSON or CSV for transition assessment, validation, and recordkeeping. This will help users as they transition from Classic Central to Central

- Created index.html with theme detection and loading of assets
- Added token.json for API configuration with placeholders
- Added token.json.example as a template for users to configure their access token
Comment thread transition/alert-inventory/alert_inventory/ui.py Fixed
Comment thread transition/alert-inventory/alert_inventory/ui.py Fixed
Comment thread transition/alert-inventory/alert_inventory/ui.py Fixed
Comment thread transition/alert-inventory/alert_inventory/classic_types.py Fixed
Comment thread transition/alert-inventory/alert_inventory/extractor.py Fixed
Comment thread transition/alert-inventory/alert_inventory/extractor.py Fixed
status_code, retry_after = _http_error_facts(error)
try:
error.close()
except Exception:
Comment thread transition/alert-inventory/alert_inventory/ui.py Fixed
@lucasvoronhpe
lucasvoronhpe self-requested a review August 30, 2026 02:38
@lucasvoronhpe lucasvoronhpe self-assigned this Aug 30, 2026

@lucasvoronhpe lucasvoronhpe left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Ran all existing python executive commands. Only have 2 general notes. In the readme, there are alternative instructions to create a token.json file using a cp command, but the token.json file already exists included in this PR. Additionally, when running the UI, I am not seeing the proper CSS styling appear in the UI. I have attached screenshots for the output I got when running the install requirements and the UI startup within the venv. I believe there may be a misnamed static css file that is trying to be rendered in this line:

Outside of the styling elements not rendering for me, I tested several functionalities that were successful including search filtering, light/dark mode, downloaded csv and json outputs. Once that is fixed, should be good to go. I can re-review for UI bugs after the CSS file naming has been corrected if you like!

Image Image

@KarthikSKumar98

Copy link
Copy Markdown
Collaborator Author

Ran all existing python executive commands. Only have 2 general notes. In the readme, there are alternative instructions to create a token.json file using a cp command, but the token.json file already exists included in this PR. Additionally, when running the UI, I am not seeing the proper CSS styling appear in the UI. I have attached screenshots for the output I got when running the install requirements and the UI startup within the venv. I believe there may be a misnamed static css file that is trying to be rendered in this line:

Outside of the styling elements not rendering for me, I tested several functionalities that were successful including search filtering, light/dark mode, downloaded csv and json outputs. Once that is fixed, should be good to go. I can re-review for UI bugs after the CSS file naming has been corrected if you like!

Image Image

Thanks for catching that. I missed a CSS file. It has been added now!

@lucasvoronhpe lucasvoronhpe left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

CSS fix is functioning as intended, all other Python commands are working as well. Looks good to merge from my end after testing on our internal ADL.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants