Added Active as a usage type to CombatStyle

[Lector]

Closes /issues/170

Hier klappt noch irgendetwas nicht. Ich weis nicht wo der Fehler ist.

[Lector]

Bei mir lokal kommt noch ein Fehler bei IdentifierGroup.ts. Ich kann aber nicht nachvollziehen wie es dazu kommt.

[elyukai]

Sehr nice, danke!

[elyukai]

Ich hätte den Namen usage_type einfach in beiden Fällen belassen. Gibt es einen Grund, warum du die Namen verändert hast bzw. unterschiedlich gemacht hast?

[Lector]

dann hätte wir in einer Datei 2 Mal usage_type ... da hätte ich jetzt einen Fehler erwartet. Wenn wir den usage_type von CombatSA und CombatStyle aufteilen müssen wir durch einen unterschiedlichen Namen doch zwischen denen unterscheiden können.

[elyukai]

Ach so, also du kannst die ja trotzdem anders in der Datei nennen, aber den Key trotzdem gleich lassen. In source/PublicationRef.ts gibt es zum Beispiel auch src und optionalSrc. Beide resultieren aber am Ende in dem gleichen Key src. Das kannst du in _ActivatableSelectOptions.ts:71 sehen.

[Lector]

Umbenennung sollte jetzt passen.

[Lector]

Da dies her ein breaking change ist kann ich gerne nach informeller Freigabe diesen PR vor dem Merge schon hier nutzen und dort alles Fixen: https://github.com/elyukai/optolith-data/pull/1229
Dann könnten beide PRs direkt nacheinander gemerged werden und es wäre schon alles repariert.

[elyukai]

Danke!

[elyukai]

Da dies her ein breaking change ist kann ich gerne nach informeller Freigabe diesen PR vor dem Merge schon hier nutzen und dort alles Fixen: elyukai/optolith-data#1229 Dann könnten beide PRs direkt nacheinander gemerged werden und es wäre schon alles repariert.

Das wäre super! Dann warte ich auf den PR mit dem Mergen hier.

[Lector]

Im data-PR wird dieses schema jetzt bereits genutzt und kann direkt mit gemerged werden.