Skip to content

Conversation

@vicgp-te
Copy link

Description

Please include a summary of the change or which issue is fixed.
Adding get_incident_log_entries function for incident timeline data to allow getting incident log entries.

  • Retrieve complete incident timeline including acknowledgments and status changes
  • Add LogEntry, LogEntryAgent, and LogEntryChannel models
  • Include comprehensive unit tests with 100% coverage
  • Update documentation and tool registration

Issue number: #45

Type of change

  • New feature (non-breaking change which adds functionality)
  • Fix (non-breaking change which fixes an issue)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation only

Checklist

  • I have performed a self-review of this change
  • Changes have been tested
  • Changes are documented
  • Changes generate no new warnings
  • PR title follows conventional commit semantics

If this is a breaking change 👇

  • I have documented the migration process
  • I have implemented necessary warnings (if it can live side by side)

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

- Retrieve complete incident timeline including acknowledgments and status changes
- Add LogEntry, LogEntryAgent, and LogEntryChannel models
- Include comprehensive unit tests with 100% coverage
- Update documentation and tool registration
@vicgp-te vicgp-te marked this pull request as ready for review September 19, 2025 13:41
@vicgp-te
Copy link
Author

@imjaroiswebdev Any chance you can review this? we would like to have this feature included to start using this tool. Thanks

@vicgp-te
Copy link
Author

vicgp-te commented Oct 9, 2025

@t1agob @jcorte-real Any change his can be reviewed? we would like to have this feature included to start using this tool. Thanks

@carlyou
Copy link

carlyou commented Oct 18, 2025

@vicgp-te my PR is kinda in the same boat here.
just in case, feel free to check out https://github.com/carlyou/pagerduty-mcp-community and add desired tools as needed.

@vicgp-te
Copy link
Author

@carlyou this PR? #26
I see you are adding list_alerts and get_alert which is different to what I am adding here. Or, are you talking about a different PR?

@vicgp-te
Copy link
Author

@blagh Any chance this PR can be reviewed by someone? It has been waiting for more than a month, or is this project maybe abandoned?

@carlyou
Copy link

carlyou commented Nov 1, 2025

@carlyou this PR? #26 I see you are adding list_alerts and get_alert which is different to what I am adding here. Or, are you talking about a different PR?

@vicgp-te I was referring to the forked repo -- feel free to send a PR there in the meanwhile.

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.

2 participants