Skip to content

Conversation

@commonly-ts
Copy link
Contributor

Description

The body when needs to be wrapped in gameJoinRestriction which somehow wasn't an issue when I did manual testing

Fixes #

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)

Changes Made

  • Wraps body inside of gameJoinRestriction in the PATCH request

  • All existing tests pass (pnpm test)

  • Added new tests for new functionality

  • Tested manually (describe below)

Manual Testing

None done, the PATCH request aligns with Roblox's documentation.

Code Quality

  • My code follows the project's coding style
  • I have run pnpm lint and fixed any issues
  • I have run pnpm format to format my code
  • I have run pnpm typecheck and there are no type errors
  • I have reviewed my own code

Documentation

  • I have updated the relevant documentation (if applicable)
  • I have added/updated JSDoc comments for new/modified code

Additional Context

Checklist

  • I have read the CONTRIBUTING.md guide
  • My PR title follows the Conventional Commits format
  • I have rebased my branch on the latest main branch
  • I have tested my changes against the Roblox Open Cloud API (if applicable)
  • This PR is ready for review

Oversight when I originally wrote this
@commonly-ts
Copy link
Contributor Author

I suspect Roblox made internal changes which makes not wrapping the body in gameJoinRestriction return 400

@codecov
Copy link

codecov bot commented Nov 20, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copy link
Member

@marinofranz marinofranz left a comment

Choose a reason for hiding this comment

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

Good eye, let's merge this 🎉

@marinofranz marinofranz merged commit 638a4e7 into relatiocc:main Nov 20, 2025
8 checks passed
github-actions bot pushed a commit that referenced this pull request Nov 20, 2025
## [1.10.2](v1.10.1...v1.10.2) (2025-11-20)

### Bug Fixes

* **universes:** update PATCH request body to include gameJoinRestriction ([#8](#8)) ([638a4e7](638a4e7))
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