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

Add option to request Items. #74

Open
Lazytron opened this issue Mar 22, 2021 · 3 comments
Open

Add option to request Items. #74

Lazytron opened this issue Mar 22, 2021 · 3 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@Lazytron
Copy link

It would be nice to extend the socket info to be able to include items on actors.

The UI would probably be a cluster on the GM side, but being able to pack an item object (or an array of them) in the socket data would be nice for macros (both player calls and environmental items).

If the item object is passed, the user side UI shouldn't be too bad as just another banner on the popup.

@akrigline
Copy link
Member

What is the use case for this? The UI pain alone makes me reluctant to do this.

@Lazytron
Copy link
Author

Mostly it's a means to work around About Time delayed effects triggering on the GM rather than the user client. Passing item dialogs for those delayed things back to the players is the idea. (hence why im less bothered about the GM side UI actually being included).

I could see uses for consumables like ropes and rations and things as well, though admittedly there are probably better ways to manage those if you mean to track them regularly.

@akrigline akrigline added enhancement New feature or request help wanted Extra attention is needed labels Mar 22, 2021
@akrigline
Copy link
Member

I won't say I get it, but providing an API to request item rolls is a fair thing to add into LMRTFY. I certainly won't turn down a PR that adds this in as long as it doesn't affect current the featureset. It will also need to make sure and keep any system-specific assumptions blocked off from general assumptions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants