Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove checkDeveloperProductName reference from addDeveloperPoduct (redundant and deprecated) #814

Merged
merged 4 commits into from
Aug 1, 2024

Conversation

alanbixby
Copy link
Member

@alanbixby alanbixby commented Jul 18, 2024

Sending a duplicate name now returns an HTTP 400 with the response

{"errorCode":"DuplicateProductName","errorMessage":"The request parameter name was invalid. Developer product name already exists in the universe.","field":"name","hint":null}

As such, Roblox deprecated checkDeveloperProductName and it is now 404'ing, thus breaking addDeveloperProduct - hence its removal.

The apis. endpoint also has a different return type which was not documented in the previous PR, this is not a breaking change as the migration is already in the library.


This change still needs better error handling + testing to be done. The PR is entered as a draft for other's reference.

@alanbixby alanbixby added help wanted Extra attention is needed and removed help wanted Extra attention is needed labels Jul 18, 2024
@alanbixby alanbixby marked this pull request as ready for review August 1, 2024 23:06
@Neztore Neztore added major Increment the major version when merged release Create a release when this pr is merged labels Aug 1, 2024
@Neztore Neztore merged commit d44b9eb into noblox:master Aug 1, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed major Increment the major version when merged release Create a release when this pr is merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants