search

jopeek / fvtt-loot-sheet-npc-5e

FVTT - Loot Sheet NPC 5E
MIT License
44 stars 86 forks source link

Feature Request: Allow GM to remove "Generate Loot Button" from player settings #320

Closed davidhawbecker closed 1 week ago

davidhawbecker commented 2 years ago

Man, this is a popular module. I don't envy how busy you are with this and I sincerely appreciate your time and effort. This is definitely one of the game change modules for foundry. So far I love everything you're doing, I just have one small request if it's at all feasible. I'm not sure why the players would need a button for "Generate loot for the selected token" and I know once I enable this, they will find it and they will ask about it and of course, try to use it, which I know they cant with their permissions, but that why I'm not sure they have access to it in the first place. Again, not a big deal, but I figure it couldn't hurt to ask. I really love your stuff man. Keep it up.

Gvicross commented 2 years ago

Has the module worked for you? My players haven't been able to buy or loot anything.

davidhawbecker commented 2 years ago

Honestly I only really use lootsheet for the loot generator/seeder function and then I use Item Piles for the players to loot bodies and containers. For shops I use either Merchant Sheet or Monks Enhanced Journal.

On Sat, Mar 5, 2022, 7:36 AM Gvicross @.***> wrote:

Has the module worked for you? My players haven't been able to buy or loot anything.

— Reply to this email directly, view it on GitHub https://github.com/jopeek/fvtt-loot-sheet-npc-5e/issues/320#issuecomment-1059756347, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATBKBGJBUIW6QJVBLDJDFQ3U6NIMZANCNFSM5P6Y6VTA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID: @.***>

DanielBoettner commented 2 years ago

Has the module worked for you? My players haven't been able to buy or loot anything.

Please check the things mentioned in the linked comment (an following):

https://github.com/jopeek/fvtt-loot-sheet-npc-5e/issues/303#issuecomment-1047525243

DanielBoettner commented 2 years ago

Man, this is a popular module. I don't envy how busy you are with this and I sincerely appreciate your time and effort. This is definitely one of the game change modules for foundry. So far I love everything you're doing, I just have one small request if it's at all feasible. I'm not sure why the players would need a button for "Generate loot for the selected token" and I know once I enable this, they will find it and they will ask about it and of course, try to use it, which I know they cant with their permissions, but that why I'm not sure they have access to it in the first place. Again, not a big deal, but I figure it couldn't hurt to ask. I really love your stuff man. Keep it up.

In short, I will look into it. The buttons should not be visible to players. Will check why they see it. Don't know how fast I get to it.

TRawss commented 2 years ago

I can't get the "generate loot" button removed from my player's view either.