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

PF2e System - Perception request rolls #136

Open
rgwynnjr opened this issue Jan 7, 2022 · 5 comments
Open

PF2e System - Perception request rolls #136

rgwynnjr opened this issue Jan 7, 2022 · 5 comments

Comments

@rgwynnjr
Copy link

rgwynnjr commented Jan 7, 2022

Perception request rolls not pulling up the dialog box for Perception and only does a public roll.

@Sasmira
Copy link

Sasmira commented Jan 12, 2022

Hello Hello,

same problem for me and it's a really problem :D

S.

@vtt-lair
Copy link
Contributor

vtt-lair commented Sep 2, 2022

Sorry I've taken so long in replying. Life has been hectic. I don't really play PF2e myself and have implemented what I could figure out at the time. I'll have a look at this once v10 is a stable version... as I assume the PF2e rolls will probably mostly break after v10.

@johncarney
Copy link

It's not just perception rolls. No requested rolls respect the visibility modifier.

@johncarney
Copy link

If it helps. here is the console output when the user rolls:

Foundry VTT | Rendering LMRTFYRoller foundry.js:5099:15
pf2e-dorako-ui | renderApplication | renderLMRTFYRoller base-theme-hooks.js:5:11
Object { app: {…} }
base-theme-hooks.js:6:11
pf2e-dorako-ui | renderLMRTFYRoller | is .window-app => add .dorako-ui base-theme-hooks.js:89:11
Foundry VTT | Registered callback for preCreateChatMessage hook foundry.js:610:13
Error: StatisticModifier#name has been split into #slug and #label
Deprecated since Version 4.1.0
Backwards-compatible support will be removed in Version 4.5.0
    logCompatibilityWarning commons.js:1692
    get name main.bundle.js:1
    _makeRoll roller.js:286
    _onSkillCheck roller.js:541
    jQuery 9
    activateListeners roller.js:168
    _render foundry.js:5138
    render foundry.js:5067
    onMessage lmrtfy.js:318
    emit index.mjs:136
    value socket.js:278
    value socket.js:265
    value socket.js:235
    emit index.mjs:136
    value manager.js:200
    emit index.mjs:136
    value index.js:119
    value manager.js:192
    emit index.mjs:136
    value socket.js:323
    emit index.mjs:136
    value transport.js:104
    value transport.js:97
    onmessage websocket.js:79
commons.js:1703:34
Rolling skill checks via actor.system.skills.med.roll() is deprecated, use actor.skills.medicine.check.roll() instead main.bundle.js:1:98340
Foundry VTT | Unregistered callback for preCreateChatMessage hook foundry.js:661:13
Foundry VTT | Created ChatMessage with id [nviHZQuWpH5qDzFI] commons.js:6946:20
[USO] Button detection aborting due to timeout. injected.js:17:291516

@johncarney
Copy link

PS. Absolutely no shame if you don't have time to maintain PF support, but please remove the PF1E/PF2E systems so you don't waste other peoples' time.

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

No branches or pull requests

4 participants