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.

Sign up for GitHub

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

Jump to bottom

Mongoose Traveller 2e: Bug in Roll for Psionic power #10094

Open
Aegidi opened this issue Jan 27, 2022 · 0 comments
Open

Mongoose Traveller 2e: Bug in Roll for Psionic power #10094

Aegidi opened this issue Jan 27, 2022 · 0 comments
Labels
bug A bug found in the character sheet

Comments

@Aegidi
Copy link

Aegidi commented Jan 27, 2022

roll20-character-sheets/Mongoose_Traveller2e/MongooseTraveller.html

Line 2430 in 9b1842f

<td style="width:130px"><button class="sheet-button" name="roll_PsionicStrength" type="roll" value='&{template:skill-attrib} {{skill=^{psionicstrength-u}}} {{character= @{character_name} }} {{roll= [[@{ro_default_rolltype} + @{mod-PsionicStrength} @{ro_default_mod}]] }} }}' ><div class="sheet-button_header" data-i18n="psionicstrength-u">Psionic Strength</div></button></td>

This line - as the only line I've seen - opens the roll template before accessing the ro_default_rolltype template. It looks to me like the default roll type also includes the "[[". This results in bugged behavior for this one button. Delete the two characters in col. 223 and 224 and you should be good.
@Aegidi Aegidi changed the title Bug in Roll for Psionic power Mongoose Traveller 2e: Bug in Roll for Psionic power Jan 27, 2022
@nmbradley nmbradley added the bug A bug found in the character sheet label Jun 19, 2023
brookesi added a commit to brookesi/roll20-character-sheets that referenced this issue Jan 23, 2024
@brookesi
- Add DD (destructive) support to all weapons on all sheets (was only…
0eabe1f 
… on vehicle and ship weapons)

- Freight lot sizes incorrectly implemented, fixed
- Add the Psi Stat to the NPC sheet Done, straight copy'n'paste but visibility controlled by checkbox
- Adding optional wounding rules from Traveller Companion New config checkbox that shows a textbox under main stats
- Under the Ship Sheet added Mortgage, Life Support, Salaries and Full Tank Fuel costs
- Parse UWPs in the form UWP: B 5 4 6 8 4 9 – 9 to allow easier copy paste from TravellerMap
- Mark GitHub Bug: Mongoose Traveller 2e: Bug in Roll for Psionic power Roll20#10094 as complete To do on release
- Added restyled roll templates from Pat, config checkbox added to control this, also tested in dark mode!
@brookesi brookesi mentioned this issue Jan 23, 2024
Closed
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug A bug found in the character sheet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nmbradley @Aegidi