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

Marshal Class #652

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

Marshal Class #652

wants to merge 7 commits into from

Conversation

Sagenlicht
Copy link
Contributor

@Sagenlicht Sagenlicht commented Nov 13, 2021

Known Issues:

  • Forgot to add AddSpellTeleportPrepareStandard and AddSpellTeleportReconnectStandard to the auras (done)
  • spell end needs review

Used Workarounds

  • As feats can't be marked as bonus feats only, I used args to handle new aura learning. Revisited once Class Features is availible
  • Spell Caster Level is discarded by the engine and set to 0. I assume because the Marshal can't cast spells and I just use the spell engine to handle the Auras and the Adrenaline Boost. Spell Level is now manually set in the "spells"

ToDo

  • Particle effects are not completed
  • Help file needs a revisit (done)

@doug1234
Copy link
Collaborator

Very nice! I think you should update py00416standard_equipment_chest.py for the marshal. Will look over it a bit more when I get a chance.

@Sagenlicht
Copy link
Contributor Author

Very nice! I think you should update py00416standard_equipment_chest.py for the marshal. Will look over it a bit more when I get a chance.

Ty @doug1234
Will take a look at the standard equipment chest script, ty for pointing this out to me.

Finished Help File
Added Marshal to py00416standard_equipment_chest.py (used fighter starting equipment)
Added teleport hooks to sp_marshal pys
regenerated co8infra + tpgamefiles.dat
Minor fixes
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

Successfully merging this pull request may close these issues.

2 participants