Skip to content

ignore: Temporary ping URL for Sharepoint integration test - #67

Closed
KevinJBoyer wants to merge 1 commit into
mainfrom
kb/temp-ping
Closed

ignore: Temporary ping URL for Sharepoint integration test#67
KevinJBoyer wants to merge 1 commit into
mainfrom
kb/temp-ping

Conversation

@KevinJBoyer

@KevinJBoyer KevinJBoyer commented Oct 29, 2025

Copy link
Copy Markdown

Preview environment for frontend

♻️ Environment destroyed ♻️

Preview environment for app

♻️ Environment destroyed ♻️

Copilot AI review requested due to automatic review settings October 29, 2025 14:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 adds a new endpoint to handle SharePoint update notifications. The endpoint provides a simple acknowledgment mechanism for when SharePoint sends update events.

  • Adds a new /sharepoint-updated GET endpoint that returns a success response

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

Comment thread app/src/app.py
Comment on lines +33 to +34
logging.info("Sharepoint updated ping.")
return {"status": "ok", "detail": "Sharepoint update received."}

Copilot AI Oct 29, 2025

Copy link

Choose a reason for hiding this comment

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

Corrected spelling of 'Sharepoint' to 'SharePoint'.

Suggested change
logging.info("Sharepoint updated ping.")
return {"status": "ok", "detail": "Sharepoint update received."}
logging.info("SharePoint updated ping.")
return {"status": "ok", "detail": "SharePoint update received."}

Copilot uses AI. Check for mistakes.
Comment thread app/src/app.py
Comment on lines +33 to +34
logging.info("Sharepoint updated ping.")
return {"status": "ok", "detail": "Sharepoint update received."}

Copilot AI Oct 29, 2025

Copy link

Choose a reason for hiding this comment

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

Corrected spelling of 'Sharepoint' to 'SharePoint'.

Suggested change
logging.info("Sharepoint updated ping.")
return {"status": "ok", "detail": "Sharepoint update received."}
logging.info("SharePoint updated ping.")
return {"status": "ok", "detail": "SharePoint update received."}

Copilot uses AI. Check for mistakes.
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