Skip to content
This repository has been archived by the owner on Jan 15, 2024. It is now read-only.

(no ticket): [update] Marketing, coupon name must be unique #1537

Merged
merged 4 commits into from
Dec 15, 2023
Merged

(no ticket): [update] Marketing, coupon name must be unique #1537

merged 4 commits into from
Dec 15, 2023

Conversation

gslandtreter
Copy link
Contributor

Creating a second coupon with the same name as an existing coupon will cause the API to return 409 Conflict. Hence, the name parameter must be unique.

Creating a second coupon with the same name as an existing coupon will cause the API to return `409 Conflict`. Hence, the name parameter must be unique.
@bc-tgomez
Copy link
Collaborator

Thanks for the feedback, @gslandtreter! We'll merge this after getting approval from @bc-traciporter. 🙇

bc-traciporter
bc-traciporter previously approved these changes Dec 12, 2023
Copy link
Contributor

@bc-traciporter bc-traciporter left a comment

Choose a reason for hiding this comment

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

I left one comment. You can take it or leave it.

reference/marketing.v2.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@slsriehl slsriehl left a comment

Choose a reason for hiding this comment

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

re-approving after accepting @bc-traciporter 's suggestion.

@slsriehl slsriehl changed the title Updated coupon name description (no ticket): [update] Marketing, coupon name must be unique Dec 15, 2023
@slsriehl slsriehl merged commit 8a0f363 into bigcommerce:main Dec 15, 2023
2 checks passed
@gslandtreter gslandtreter deleted the patch-1 branch December 18, 2023 19:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
external ready for review This PR is complete and awaiting review ready-to-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants