Skip to content

Added cantrips and blessings to activatable prereqs#173

Open
Lector wants to merge 7 commits into
mainfrom
feature/prereq-cantrip
Open

Added cantrips and blessings to activatable prereqs#173
Lector wants to merge 7 commits into
mainfrom
feature/prereq-cantrip

Conversation

@Lector

@Lector Lector commented Jun 23, 2026

Copy link
Copy Markdown
Collaborator

Closes #172

@Lector Lector requested a review from elyukai as a code owner June 23, 2026 19:21

@elyukai elyukai left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@Lector

Lector commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator Author

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 thread src/types/_IdentifierGroup.ts Outdated
comment: "Requires a specific blessing.",
type: () =>
DB.Object({
blessing: DB.Required({

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typischerweise wird der Key, der den Eintrag definiert, id genannt. Zumindest bei den anderen Voraussetzungen.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Du kannst aber trotzdem einen Display Name für die Property verwenden, wenn er im Editor anders heißen soll.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich hätte jetzt grundsätzlich BlessingPrerequisite und CantripPrerequisite unter TinyActivatablePrerequisite zusammengefasst, weil beide gleich funktionieren. Dann kannst du eine TinyActivatableIdentifierGroup in _IdentifierGroup.ts mit den beiden Entities erstellen und dann als ID verwenden.

Lector added 3 commits June 25, 2026 13:26
…n _IdentifierGroup.ts

Removed this Prereqs from SpecialAbilities in _IdentifierGroup.ts
Renamed identifying key to id in this 2 Prereqs
@Lector

Lector commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator Author

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.

@elyukai

elyukai commented Jun 27, 2026

Copy link
Copy Markdown
Member

Also du hast die IdentifierGroups angepasst und auch die neue hinzugefügt, aber noch nicht die Voraussetzungen selbst angepasst.

@Lector

Lector commented Jun 27, 2026

Copy link
Copy Markdown
Collaborator Author

Alles klar. Sollte jetzt passen.

@elyukai

elyukai commented Jun 27, 2026

Copy link
Copy Markdown
Member

Du hast einmal BlessingPrerequisite und einmal CantripPrerequisite. Beide funktionieren genau gleich, nur mit anderen IDs. Ich würde das analog zu anderen Voraussetzungen machen und daraus eine kombinierte Voraussetzung TinyActivatablePrerequisite machen, wo man dann in der Voraussetzung wählt, welche Entity und welche Instanz gewünscht ist. Bei ActivatablePrerequisite wählt man ja auch erst die Entity und dann die Instanz, aber es ist alles in einem einzelnen Voraussetzungstyp.

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.

Magical special ability - prereq - cantrip

2 participants