Skip to content

Added the 2 new options for endpoint type#7721

Merged
johnsimons merged 2 commits into
masterfrom
john/new-types
Aug 13, 2025
Merged

Added the 2 new options for endpoint type#7721
johnsimons merged 2 commits into
masterfrom
john/new-types

Conversation

@johnsimons

Copy link
Copy Markdown
Member

These are introduced in Particular/ServicePulse#2580

Comment thread servicepulse/usage.md Outdated

#### Gateway or Bridging Endpoint

This is an infrastructure endpoint used by the Particular Platform<

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should these < be .?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Given that these have the same descriptions, will customers know which one to pick?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is based on https://github.com/Particular/CustomerSuccess/issues/4834, which suggests to add a single entry.
Maybe that is the right way to go, thoughts @mikeminutillo ?

My concern is that "Infrastructure Endpoint" is too generic and users can confuse it with their own endpoints that deal with infrastructure.
That said, maybe we can be more explicit and name it "Particular Platform Endpoint"?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't mind having them separate, but their descriptions should maybe provide more detail as to what fits in each category.

The Particular Platform Endpoints might explicitly mention Error, Audit, and Monitoring queues, as well as the ServiceControl instance input queues.

The Gateway or Bridging Endpoints might link off to the documentation for Gateway and Bridge (if links are possible)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

ok I'll update these

@johnsimons

Copy link
Copy Markdown
Member Author

@mikeminutillo I have updated this, please review again.
I have also updated Particular/ServicePulse#2580

@johnsimons johnsimons merged commit 0cabdd9 into master Aug 13, 2025
4 checks passed
@johnsimons johnsimons deleted the john/new-types branch August 13, 2025 01:04
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