Skip to content

Conversation

@alon-boshi
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings January 26, 2026 08:51
@linear
Copy link

linear bot commented Jan 26, 2026

@netlify
Copy link

netlify bot commented Jan 26, 2026

Deploy Preview for permitio-docs ready!

Name Link
🔨 Latest commit 1237f85
🔍 Latest deploy log https://app.netlify.com/projects/permitio-docs/deploys/697934afe376fb0007b4bf38
😎 Deploy Preview https://deploy-preview-609--permitio-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR enhances the Monitoring PDPs documentation by providing comprehensive guidance on the PDP Monitoring page, its purpose, and how to interpret connection statuses. The documentation has been restructured to clarify operational expectations and address common misconceptions about PDP status indicators.

Changes:

  • Updated sidebar configuration to use explicit doc type with a custom label
  • Completely rewrote the monitoring PDPs documentation to provide detailed operational guidance
  • Added sections explaining connection status interpretation, common scenarios for red PDPs, and version management

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
sidebars.js Changed sidebar entry from simple string reference to explicit doc type configuration with custom label
docs/how-to/monitoring-pdps/monitoring-pdps.mdx Comprehensive rewrite of monitoring documentation including EAP notice, connection status guidance, timeout configuration, and version management best practices

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@alon-boshi alon-boshi requested a review from zeevmoney January 26, 2026 09:01
@argos-ci
Copy link

argos-ci bot commented Jan 26, 2026

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build Status Details Updated (UTC)
default (Inspect) 👍 Changes approved 54 changed, 1 added Jan 27, 2026, 10:09 PM

@zeevmoney zeevmoney requested a review from danyi1212 January 27, 2026 21:31
![Monitoring Page](/images/monitoring/monitoring-page.png)
# Monitoring PDPs
:::note Early Access Program
The PDP Monitoring page is an **Early Access Program (EAP)** feature.
Copy link
Contributor

Choose a reason for hiding this comment

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

???

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Omer told me to add EAP here

Copy link
Contributor

@zeevmoney zeevmoney left a comment

Choose a reason for hiding this comment

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

The Monitoring Page retrieves data via the [PDP Statistics API](./../../api/pdp-statistics.mdx), ensuring up-to-date insights.

![Monitoring Page](/images/monitoring/monitoring-page.png)
# Monitoring PDPs
Copy link
Contributor

Choose a reason for hiding this comment

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

Add frontmatter at the beginning of the file:

---
sidebar_position: 1
title: Monitoring PDPs
---

# Monitoring PDPs

Copy link
Contributor Author

Choose a reason for hiding this comment

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

will do

Comment on lines +138 to +141
- [PDP Statistics API](../../api/pdp-statistics) — Programmatic access to PDP monitoring data
- [PDP Webhooks](../../api/pdp-webhooks) — Real-time notifications for PDP events
- [PDP Overview](../../concepts/pdp/overview) — Learn more about Policy Decision Points
- [PDP Configuration](../../concepts/pdp/configuration) — Configure PDP settings and timeouts
Copy link
Contributor

Choose a reason for hiding this comment

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

Use absolute paths from the docs root for more resilience:

  • /concepts/pdp/overview instead of ../../concepts/pdp/overview
  • /concepts/pdp/configuration instead of ../../concepts/pdp/configuration
  • /api/pdp-statistics instead of ../../api/pdp-statistics
  • /api/pdp-webhooks instead of ../../api/pdp-webhooks

Copy link
Contributor Author

Choose a reason for hiding this comment

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

will do

@zeevmoney
Copy link
Contributor

@alon-boshi I don't see the page in the preview, what is the path for it?

@alon-boshi
Copy link
Contributor Author

"how-to/monitoring-pdps/monitoring-pdps",
{
type: "doc",
id: "how-to/monitoring-pdps/monitoring-pdps",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
id: "how-to/monitoring-pdps/monitoring-pdps",
id: "how-to/monitoring-pdps",

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