Wird das script irgenwo benutzt außer in der InventoryDependentInteractable test scene?
Aktuell sind noch ein paar Sachen kaputt. Daher würde ich es noch nicht mergen. Ggf. mergen wir das erst nach der Gamescom und machen aus dem aktuellen Branch nach Bedarf Builds.
Yelis Dialog ist gerade komplett kaputt. Bei mir zumindest wird nie "Please could you get the runner ducks back into their coop?" erreicht und auch nicht die zweite Quest aktiviert. Kannst du den gesamten Questdurchlauf bei dir mal gegenchecken?
Durch die Änderung wird die erste Quest nicht richtig getriggert und das Post-it zeigt nur den Standardtext.
Ich habe noch einmal darüber nachgedacht und bin mir sehr sicher, dass wir damit in eine cyclic dependency geraten, sobald eine Tomatenpflanze nicht nur Tomaten, sondern auch selbst wieder Samen droped. Selbst wenn das Gamedesign technisch nicht vorgesehen ist, schaffen wir uns hier einen ganz gefährlichen Sonderfall.
Ich musste im Code nachschauen, um zu verstehen, wofür das überhaupt ist. Ich denke, das braucht mindestens einen anderen Namen.
CallGroup hat ziemlich viel codesmell. Kannst du gerne für jetzt so lassen, aber setz da am besten noch ein To-do hinter, damit wir daran denken, das zu ändern, sobald wir eine event bus implementierung haben.
Nen change ohne changes. Währe etwas sauberer, wenn hier garkein change währe
Yelis Dialog ist gerade komplett kaput. Bei mir zumindest wird nie "Please could you get the runner ducks back into their coop?" erreicht und auch nicht die zweite quest aktiviert
Ich habe noch einmal darüber nachgedacht und bin mir sehr sicher, dass wir damit in eine cyclic dependency geraten, sobald eine Tomatenpflanze nicht nur Tomaten, sondern auch selbst wieder Samen…
9bebe1a44d
Merge pull request 'Implemented first demo quest line' (#15) from feature/implement_quests into develop