[FRHOF][FRAIF] Full Book Content Upload#335
[FRHOF][FRAIF] Full Book Content Upload#335tregolani wants to merge 16 commits intoAuroraLegacy:masterfrom
Conversation
Companions, Deities, Languages, and Spells
FreshRoyal
left a comment
FreshRoyal
left a comment
There was a problem hiding this comment.
Code looks good to me so far, but I did leave some comments. I'll do a more thorough review once I'm done looking over DMG24.
|
|
||
| <element name="Cult of the Dragon Initiate" type="Feat" source="Forgotten Realms: Heroes of Faerûn" id="ID_WOTC_FRHOF_FEAT_CULT_OF_THE_DRAGON_INITIATE"> | ||
| <supports>Origin</supports> | ||
| <prerequisite>Origin Feat, Forgotten Realms Campaign</prerequisite> |
There was a problem hiding this comment.
We didn't add "Origin Feat" to the prerequisites in PHB24
| <description>You gain the following benefits.</description> | ||
| </sheet> | ||
| <rules> | ||
| <select type="Language" name="Language Proficiency (Cult of the Dragon Initiate)" supports="Standard||Exotic" default="ID_WOTC_PHB24_LANGUAGE_DRACONIC" /> |
There was a problem hiding this comment.
Are "Rare" and "Exotic" languages used interchangeably for backwards compatibility?
There was a problem hiding this comment.
I need to check on this one. I likely just copied some code from a language grant. It's possible we need to add "Standard||Rare||Exotic" instead.
EDIT: Looks like PHB24 only uses "Rare" to denote languages either originating from another plane or secret (Druidic/Thieves' Cant). It does not use the "Exotic" or "Secret" categories at all—these were exclusive to PHB14 and its supplements. From the look of it, PHB24 "Rare" is equivalent to PHB14 "Exotic + Secret"
| </sheet> | ||
| <rules> | ||
| <select type="Language" name="Language Proficiency (Cult of the Dragon Initiate)" supports="Standard||Exotic" default="ID_WOTC_PHB24_LANGUAGE_DRACONIC" /> | ||
| <select type="Feat Feature" name="Dragon Terror" supports="Dragon Terror" default="ID_WOTC_FRHOF_FEAT_FEATURE_CULT_OF_THE_DRAGON_INITIATE_DRAGONS_TERROR" /> |
There was a problem hiding this comment.
Don't we usually just use "Feat Features" for selectable features?
There was a problem hiding this comment.
Because the feat adds benefits that are "always on" and others that are limited by uses, I created a "selectable" feature with a default so it nests the benefit under the main feat item on the sheet. Therefore I can have a usage callout for the "sub-benefit" while still listing the main benefit as "always on".
I used this "sub-benefit" styling for quite a few of the subclasses and feats.
| </element> | ||
|
|
||
| <element name="Level 3: Knightly Envoy" type="Archetype Feature" source="Forgotten Realms: Heroes of Faerûn" id="ID_WOTC_FRHOF_ARCHETYPE_FEATURE_FIGHTER_BANNERET_KNIGHTLY_ENVOY"> | ||
| <requirements>!ID_INTERNAL_FRHOF_FEATURE_REPLACEMENT_FIGHTER_BANNERET_KNIGHTLY_ENVOY</requirements> |
There was a problem hiding this comment.
Similar to my question about ABH, are we adding replacement grants to supplement subclasses, too?
There was a problem hiding this comment.
Didn't we do this for subclasses in PHB24? Are we going to treat supplement stuff differently? I'll let you guys make the call.
There was a problem hiding this comment.
Before 5.5e, it was just the subclasses that came with the class that would get replacement grants because of the OCFs Beast Master got in TCoE. The only exception I can think of was Divine Strike/Potent Spellcasting for Cleric subclasses. I'm personally okay with all supplement subclasses getting them now, but if that's the case, it would cause inconsistencies with older subclasses.
FreshRoyal
left a comment
FreshRoyal
left a comment
There was a problem hiding this comment.
Source information setters
| <set name="abbreviation">FRAIF</set> | ||
| <set name="url">https://marketplace.dndbeyond.com/category/4714000?pid=D4714000</set> | ||
| <set name="image">https://raw.githubusercontent.com/AuroraLegacy/elements/master/supplements/forgotten-realms-adventures-in-faerun/FRAIF.png</set> | ||
| <set name="author" abbreviation="WOTC" url="http://dnd.wizards.com">Wizards of the Coast</set> |
There was a problem hiding this comment.
| <set name="author" abbreviation="WOTC" url="http://dnd.wizards.com">Wizards of the Coast</set> | |
| <set name="author" abbreviation="WOTC" url="http://dnd.wizards.com">Wizards of the Coast</set> | |
| <set name="information">D&D 5.5e</set> |
| <set name="abbreviation">FRHOF</set> | ||
| <set name="url">https://marketplace.dndbeyond.com/category/4712000?pid=D4712000</set> | ||
| <set name="image">https://raw.githubusercontent.com/AuroraLegacy/elements/master/supplements/forgotten-realms-heroes-of-faerun/FRHOF.png</set> | ||
| <set name="author" abbreviation="WOTC" url="http://dnd.wizards.com">Wizards of the Coast</set> |
There was a problem hiding this comment.
| <set name="author" abbreviation="WOTC" url="http://dnd.wizards.com">Wizards of the Coast</set> | |
| <set name="author" abbreviation="WOTC" url="http://dnd.wizards.com">Wizards of the Coast</set> | |
| <set name="information">D&D 5.5e</set> |
2 participants
This PR contains all of the relevant player-facing options from the supplements Forgotten Realms: Heroes of Faerûn and Forgotten Realms: Adventures in Faerûn.