Added cantrips and blessings to activatable prereqs - #173
Conversation
elyukai
left a comment
elyukai
left a comment
There was a problem hiding this comment.
Das kannst du so einfach leider nicht implementieren. Was da drin steht hat Einfluss auf die gesamte Wahrnehmung in Optolith, was ein Activatable ist. Ein Activatable ist quasi definiert als Vorteile, Nachteile und Sonderfertigkeiten. Segnungen und Zaubertricks zählen da nicht so direkt zu, da sie keine weiteren Optionen wie Stufen und Auswahlmöglichkeiten haben. Daher wäre es denke ich sinnvoller, dafür einen neuen Voraussetzungstyp einzuführen. Ich habe Cantrips und Blessings beim Charakter-Schema als TinyActivatable zusammengefasst, das könnte man hier analog dazu machen.
Daher kommt auch ein Fehler bei den Cache-Buildern, da die dann ein Handling für die neuen Activatable-Entities erwarten.
|
Ich habe jetzt Cantrips and Blessing unter GeneralPrerequisites eingefügt. Sie haben also mit Activatable nichts mehr zu tun. Dennnoch bekomme ich noch irgendeinen Fehler den ich nicht ganz verstehe. |
| comment: "Requires a specific blessing.", | ||
| type: () => | ||
| DB.Object({ | ||
| blessing: DB.Required({ |
There was a problem hiding this comment.
Typischerweise wird der Key, der den Eintrag definiert, id genannt. Zumindest bei den anderen Voraussetzungen.
There was a problem hiding this comment.
Du kannst aber trotzdem einen Display Name für die Property verwenden, wenn er im Editor anders heißen soll.
…n _IdentifierGroup.ts Removed this Prereqs from SpecialAbilities in _IdentifierGroup.ts Renamed identifying key to id in this 2 Prereqs
…ptolith-database-schema into feature/prereq-cantrip
|
Die Punkte sind umgesetzt. Es scheint aber noch Probleme mit dem cache zu geben. Da bräuchte ich ein par Hinweise was ich da noch anpassen muss. |
|
Also du hast die IdentifierGroups angepasst und auch die neue hinzugefügt, aber noch nicht die Voraussetzungen selbst angepasst. |
|
Alles klar. Sollte jetzt passen. |
|
Du hast einmal |
|
Ist umgesetzt. |
elyukai
left a comment
elyukai
left a comment
There was a problem hiding this comment.
Ein paar kleine Änderungswünsche hab ich noch. 🙈
| DB.Object({ | ||
| id: DB.Required({ | ||
| comment: "The required item.", | ||
| type: DB.IncludeIdentifierType(TinyActivatableCategory), |
There was a problem hiding this comment.
Hier kannst du einfach direkt TinyActivatableIdentifier verwenden und brauchst TinyActivatableCategory nicht extra.
There was a problem hiding this comment.
Den TinyActivatableIdentifier gibts nicht. Auch nicht in _Identifier.ts. Soll ich den anlegen oder passts so?
There was a problem hiding this comment.
Du hast den in _IdentifierGroup.ts angelegt. 🙈
There was a problem hiding this comment.
Das wird mir hier in der PR jedenfalls so angezeigt.
There was a problem hiding this comment.
Ja ... war schon ne Weile her
There was a problem hiding this comment.
Jetzt sollte es passen
Reordered entries in PrerequisiteGroups.ts
…ptolith-database-schema into feature/prereq-cantrip
2 participants
Closes #172