Skip to content

Conversation

mjger
Copy link
Contributor

@mjger mjger commented Aug 15, 2025

No description provided.

Copy link
Contributor

@dkluever22 dkluever22 left a comment

Choose a reason for hiding this comment

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

@mjger Please take a look at the two suggestions and let me know what you think.

@@ -146,6 +146,18 @@ post:
responses:
204:

/availabilityNotifications:
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
/availabilityNotifications:
/availability-notifications:

/availabilityNotifications:
displayName: Product Availability Notification
post:
description: Modifies information on the email notification assignments of a single product by adding the email address of the customer and the storefront locale. *epagesNow
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
description: Modifies information on the email notification assignments of a single product by adding the email address of the customer and the storefront locale. *epagesNow
description: Enables the availability notification for the given product and customer email address. The language of the notification is based on the locale of the customer's storefront. *epagesNow

I would phrase the description differently because in my opinion "modifies information on the email notification assignment" is a bit abstract. Let me know what you think.

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

Successfully merging this pull request may close these issues.

2 participants