Skip to content
This repository has been archived by the owner on Apr 15, 2022. It is now read-only.

[FEATURE] Make Loot sheet with a button to actually loot #17

Open
2 tasks
greenlink opened this issue Sep 17, 2020 · 5 comments
Open
2 tasks

[FEATURE] Make Loot sheet with a button to actually loot #17

greenlink opened this issue Sep 17, 2020 · 5 comments
Labels
enhancement New feature or request low priority

Comments

@greenlink
Copy link

I'm really annoyed by the fact that the players can't take the loot when the sheet is with a limited permission.

  • Create a button under the avatar image to take all the loot of the Loot Sheet.
  • Create a button in each item to take the loot of that row.
@greenlink greenlink added the enhancement New feature or request label Sep 17, 2020
@Djphoenix719
Copy link
Owner

This isn't possible. Players are unable to edit sheets they do not own - it is baked into Foundry's permissions system.

@Djphoenix719 Djphoenix719 added the wontfix This will not be worked on label Sep 17, 2020
@greenlink
Copy link
Author

I think you are a little bit mistaken, because there is a module called "Loot Sheet to 5e" that does that, i think the sheet is in observer permission in reality, so I was mistaken too. But I would really love those functionalitys that I mentioned. Could you have a look on that module?

@Djphoenix719
Copy link
Owner

The way loot sheet for 5e does it is by sending a request to the GM to do so, which the module running on the GM then does. So while you are correct in that it is possible, I am still cautious as to it being necessary.

Can you describe a situation where you need this solution as opposed to just giving the players owner permissions?

@Djphoenix719 Djphoenix719 reopened this Sep 21, 2020
@Djphoenix719 Djphoenix719 added question Further information is requested and removed wontfix This will not be worked on labels Sep 21, 2020
@greenlink
Copy link
Author

greenlink commented Sep 21, 2020

The loot all button would distribute all Items in the sheet of the player character automatically (it's better than drag one by one), the loot item button would do the same but for that item. The best about both of them is that you don't need to open your character sheet.
If that mechanism is implemented, you will also be able to make the merchant sheet to actually sell items, exchanging the gold pieces and putting the bought itens on the player sheet. Indeed like the Loot Sheet to 5e module. But that depends if you would like that to be a tool on your toolbox.

@Djphoenix719
Copy link
Owner

Hmmm alright. I suppose that's valid for convenience's sake. No promises on this but I understand the use case for it. Mostly a QOL improvement.

@Djphoenix719 Djphoenix719 added low priority and removed question Further information is requested labels Sep 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request low priority
Projects
None yet
Development

No branches or pull requests

2 participants