Skip to content

Knowledge subskills

M.A. requested to merge mkahvi/foundryvtt-pathfinder1:knowledge-subskills into master

Changes knowledge skills to be subskills of "knowledge".

Migrates old knowledge skills to the new subskills, deleting the old data.

On the side this also:

  • Initializes missing skill ranks to 0.

TODO:

  • Things broken:
    • Background skills optional rule does not split the knowledge skills correctly.
    • Journal entries are no longer linked to the knowledge skills.
    • Class skills in compendium entries
    • Adding background knowledge skills creates adventure knowledge skills instead.
  • Inconsistencies:
    • Knowledge skills have delete button.
  • Temporary backwards compatibility
    • Deprecation warnings everywhere

The long skill IDs feel a bit questionable since everything else in the system is short 3 letter codes.

Implements #756

Together with !878 (merged) this...
Implements #578

Edited by M.A.

Merge request reports