Skip to content

Conversation

@kirre-bylund
Copy link
Contributor

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for simplified inventory listing functionality for both player and character inventories. The implementation provides a more optimized way to retrieve inventory data with filtering and pagination capabilities.

Key changes:

  • Introduces new data structures for simplified inventory items and responses
  • Adds API methods for listing player and character inventories with optional filtering
  • Implements both default parameter and configurable versions of inventory listing methods

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
LootLockerSDKManager.h Declares public API methods for simplified inventory listing
LootLockerManager.h Adds Blueprint-callable wrapper methods and delegate for inventory responses
LootLockerGameEndpoints.h Registers new inventory listing endpoint
LootLockerPlayerRequestHandler.h Defines data structures and declares handler methods for inventory requests
LootLockerSDKManager.cpp Implements SDK manager methods that delegate to request handler
LootLockerManager.cpp Implements Blueprint wrapper methods with lambda callbacks
LootLockerGameEndpoints.cpp Initializes the inventory listing endpoint
LootLockerPlayerRequestHandler.cpp Implements HTTP request logic with query parameter handling

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kirre-bylund kirre-bylund merged commit 2b3027c into dev Nov 20, 2025
@kirre-bylund kirre-bylund deleted the feat/simple-inventory branch November 20, 2025 11:07
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.

3 participants