Skip to content

Social and Inventory endpoints#530

Merged
jellejurre merged 12 commits intovrchatapi:mainfrom
VinyarionHyarmendacil:inventory-and-social-endpoints-3
Dec 4, 2025
Merged

Social and Inventory endpoints#530
jellejurre merged 12 commits intovrchatapi:mainfrom
VinyarionHyarmendacil:inventory-and-social-endpoints-3

Conversation

@VinyarionHyarmendacil
Copy link
Contributor

/calendar/discover
/economy/purchase/listing
/economy/purchases
/inventory/{inventoryItemId}/consume
/inventory/{inventoryItemId}/equip
/inventory/{inventoryItemSlot}/equip
/products/{productId}
/users/{userId}/boop
/users/{userId}/mutuals
/users/{userId}/mutuals/friends
/users/{userId}/mutuals/groups

Addresses #474 and #495

Copy link
Member

@ariesclark ariesclark left a comment

Choose a reason for hiding this comment

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

Lint issues. Should've failed the check, will look into why it didn't.

@jellejurre
Copy link
Contributor

How do I get the Emoji ID for boops?

@jellejurre
Copy link
Contributor

Got my answer in the discord, updated documentation, only issue is
211:42 error path-params Paths "/inventory/{inventoryItemId}/equip" and "/inventory/{inventoryItemSlot}/equip" must not be equivalent. paths./inventory/{inventoryItemSlot}/equip

in the linting, waiting on a response from Aries on whether we ignore this or fix the rule before merging

@ariesclark
Copy link
Member

Do generators handle this issue correctly? If so, we could disable the rule.

@jellejurre jellejurre force-pushed the inventory-and-social-endpoints-3 branch 4 times, most recently from ec3cc28 to b97ab33 Compare December 3, 2025 23:34
VinyarionHyarmendacil and others added 12 commits December 4, 2025 01:01
/calendar/discover
/economy/purchase/listing
/economy/purchases
/inventory/{inventoryItemId}/consume
/inventory/{inventoryItemId}/equip
/inventory/{inventoryItemSlot}/equip
/products/{productId}
/users/{userId}/boop
/users/{userId}/mutuals
/users/{userId}/mutuals/friends
/users/{userId}/mutuals/groups

Addresses vrchatapi#474 and vrchatapi#495
Previous commit and was put on the wrong branch
@jellejurre jellejurre force-pushed the inventory-and-social-endpoints-3 branch from b97ab33 to f48b9d0 Compare December 4, 2025 00:01
Copy link
Contributor

@jellejurre jellejurre left a comment

Choose a reason for hiding this comment

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

LGTM, lets wait with merge until Aries is done with CI changes and then merge

@jellejurre jellejurre merged commit 709644f into vrchatapi:main Dec 4, 2025
5 of 6 checks passed
@VinyarionHyarmendacil VinyarionHyarmendacil deleted the inventory-and-social-endpoints-3 branch December 9, 2025 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants