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

Allow players to use result spender #51

Closed
Aethen22 opened this issue Jun 27, 2021 · 2 comments
Closed

Allow players to use result spender #51

Aethen22 opened this issue Jun 27, 2021 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@Aethen22
Copy link

Right now they cant do it because the message is sent as "Gamemaster" and they cant update a GM message(Lack of permissions, which is a good thing most of the times because we dont want players able to mess with GM messages), or at least that's what im guessing it may be.
Said button(Player who said it was Gamemaster):
image
Said error(Lacks permission, my guess is the above reason):
image

@Aethen22
Copy link
Author

A friend of mine(and player in my campaign) fixed it this way.
Sustituted line 38 of dice_helper.js to this:
if (messageData.message.user === game.userId && app.roll && (messageData.message.content.search('Initiative') === -1 || messageData.message.content.search('Help spending results') === -1 || messageData.message.content.search('for spending results') === -1)) {

@wrycu wrycu added the enhancement New feature or request label Jul 16, 2021
@wrycu wrycu changed the title Enabling players activating the "Help spending results!" button. Allow players to use result spender Jul 16, 2021
@wrycu wrycu self-assigned this Jul 16, 2021
@wrycu wrycu mentioned this issue Jul 16, 2021
13 tasks
@wrycu wrycu added this to the release-0.2.0 milestone Jul 17, 2021
wrycu pushed a commit that referenced this issue Jul 17, 2021
@wrycu
Copy link
Owner

wrycu commented Jul 17, 2021

Done, will be released in 0.2.0.

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

No branches or pull requests

2 participants