fr.jeanf.questsystem 0.0.61 → 0.0.63

package/Runtime/Quests/Quest_VisitAllPlaces.prefab

@@ -28,13 +28,13 @@ Transform:
   m_PrefabInstance: {fileID: 0}
   m_PrefabAsset: {fileID: 0}
   m_GameObject: {fileID: 5258580714634048286}
-  serializedVersion: 2
   m_LocalRotation: {x: -0, y: -0, z: -0, w: 1}
   m_LocalPosition: {x: 7.37, y: 1, z: 6.08}
   m_LocalScale: {x: 3, y: 3, z: 3}
   m_ConstrainProportionsScale: 0
   m_Children: []
   m_Father: {fileID: 0}
+  m_RootOrder: -2
   m_LocalEulerAnglesHint: {x: 0, y: 0, z: 0}
 --- !u!33 &7682207879693783699
 MeshFilter:
@@ -171,7 +171,8 @@ MonoBehaviour:
   progress: 0
   clearToStart: 0
   QuestProgress: {fileID: 11400000, guid: d6b05e1957994c8449fb72e849a2a3d6, type: 2}
-  StartQuestEventChannel: {fileID: 11400000, guid: c510c1e4d26428740b3e329cca98e5d1,
-    type: 2}
   QuestInitialCheck: {fileID: 11400000, guid: 4f5235d1ff779064cabee9ca65296d7f, type: 2}
+  resetChannel: {fileID: 11400000, guid: f5878d9ead688634e8a37cb28026e722, type: 2}
   requirementCheck: {fileID: 11400000, guid: a4dae407f780f974abbf43c280515876, type: 2}
+  loadRequiredScenesEventChannel: {fileID: 0}
+  unlockDoorsEventChannel: {fileID: 0}

package/Runtime/Scripts/Core/Quest.cs

@@ -47,37 +47,6 @@ namespace jeanf.questsystem
             this.questSO = questQuestSo;
             this.state = questState;
             this.questStepStates = questStepStates;
-
-            //if the quest step states and prefabs are different lengths,
-            //something has changed during development and the saved data is out of sync.
-            //if (this.questStepStates.Length != this.questSO.questSteps.Length)
-            //{
-            //    Debug.LogWarning("Quest Step Prefabs and Quest Step States are "
-            //                     + "of different lengths. This indicates something changed "
-            //                     + "with the QuestInfo and the saved data is now out of sync. "
-            //                     + "Reset your data - as this might cause issues. QuestId: " + this.questSO.id);
-            //}
         }
-
-
-
-        //public void StoreQuestStepState(QuestStepState questStepState, int stepIndex)
-        //{
-        //    if (stepIndex < questStepStates.Count)
-        //    {
-        //        questStepStates[stepIndex].state = questStepState.state;
-        //    }
-        //    else
-        //    {
-        //        Debug.LogWarning("Tried to access quest step data, but stepIndex was out of range: "
-        //                         + "Quest Id = " + questSO.id + ", Step Index = " + stepIndex);
-        //    }
-        //}
-
-        //public QuestData GetQuestData()
-        //{
-        //    return new QuestData(state, currentQuestStepIndex, questStepStates);
-        //}
-
     }
 }

package/Runtime/Scripts/Core/QuestItem.cs

@@ -11,6 +11,8 @@ namespace jeanf.questsystem
 {
     public class QuestItem : MonoBehaviour, IDebugBehaviour, IValidatable
     {
+
+        #region interface variables
         public bool isDebug
         {
             get => _isDebug;
@@ -20,28 +22,30 @@ namespace jeanf.questsystem
 
         [SerializeField] private bool _isDebug = false;
         [SerializeField] private bool _startQuestOnEnable = false;
+        #endregion
 
+        #region Step Dictionaries
         [Tooltip("Visual feedback for the quest state")] [Header("Quest")] 
         [SerializeField] [Validation("A reference to a questSO is required")] private QuestSO questSO;
         private Dictionary<string, QuestStep> stepMap = new Dictionary<string, QuestStep>();
         private Dictionary<string, QuestStep> activeSteps = new Dictionary<string, QuestStep>();
         private Dictionary<string, QuestStep> completedSteps = new Dictionary<string, QuestStep>();
         private List<QuestStep> rootSteps = new List<QuestStep>();
+        #endregion
 
-        public delegate void ValidateStep(string stepId);
-
-        public static ValidateStep ValidateStepEvent;
-
-
-        [ReadOnly] [Range(0, 1)] [SerializeField]
-        private float progress = 0.0f;
 
+        #region main data
         [SerializeField] [ReadOnly]
         private bool clearToStart = false;
-
         private string questId;
         private QuestState currentQuestState;
+        [ReadOnly][Range(0, 1)][SerializeField]
+        private float progress = 0.0f;
+        #endregion
 
+        #region events
+        public delegate void ValidateStep(string stepId);
+        public static ValidateStep ValidateStepEvent;
         // these events are Located in Assets/Resources/Quests/Channels - it is searched for at Awake time.
         // if they do not exist simply right click in the hierarchy and find >InitializeQuestSystem<
         [Header("Listening on:")] 
@@ -53,10 +57,10 @@ namespace jeanf.questsystem
         private StringEventChannelSO requirementCheck;
         [SerializeField][Validation("A reference to the LoadRequiredScenesEventChannel SO is required")]  StringListEventChannelSO loadRequiredScenesEventChannel;
         [SerializeField] IntEventChannelSO unlockDoorsEventChannel;
-        
-        
+        #endregion
+
 
-        #region Awake/Enable/Disable
+        #region Standard Unity Methods
         private void Awake()
         {
             questId = questSO.id;
@@ -67,7 +71,10 @@ namespace jeanf.questsystem
                 rootSteps.Add(questSO.rootSteps[i]);
             }
         }
-
+        private void Reset()
+        {
+            Init(questId);
+        }
         private void OnEnable()
         {
             Subscribe();
@@ -76,7 +83,6 @@ namespace jeanf.questsystem
         }
         private void OnDisable() => Unsubscribe();
         private void OnDestroy() => Unsubscribe();
-
         private void Subscribe()
         {
             resetChannel.OnEventRaised += Reset;
@@ -88,7 +94,6 @@ namespace jeanf.questsystem
             QuestStep.stepActive += UpdateStepStatus;
             QuestStep.childStep += AddStepToStepMap;
         }
-
         private void Unsubscribe()
         {
             resetChannel.OnEventRaised -= Reset;
@@ -104,11 +109,6 @@ namespace jeanf.questsystem
         }
         #endregion
 
-        private void Reset()
-        {
-            Init(questId);
-        }
-
         #region Instantiations & Loading
         public void InstantiateQuestStep(string id)
         {
@@ -116,10 +116,10 @@ namespace jeanf.questsystem
             if (stepMap[id].stepStatus != QuestStepStatus.Inactive) return;
             if (activeSteps.ContainsKey(id)) return;
 
+
             Instantiate(stepMap[id], this.transform, true);
-            //if(!activeSteps.ContainsKey(id)) activeSteps.Add(id,Instantiate(stepMap[id], this.transform, true));
-        }
 
+        }
         private void LoadDependencies()
         {
             loadRequiredScenesEventChannel.RaiseEvent(questSO.ScenesToLoad);
@@ -129,43 +129,22 @@ namespace jeanf.questsystem
                 unlockDoorsEventChannel.RaiseEvent(roomToUnlock);
             }
         }
-        #endregion
-
-        public void UpdateStepStatus(string id, QuestStepStatus status)
-        {
-            switch (status)
-            {
-                case QuestStepStatus.Completed when activeSteps.ContainsKey(id):
-                    // put step in completed list
-                    activeSteps.Remove(id);
-                    completedSteps.Add(id, stepMap[id]);
-                    break;
-                case QuestStepStatus.Active when !activeSteps.ContainsKey(id):
-                    // put step in active list
-                    activeSteps.Add(id, stepMap[id]);
-                    break;
-                case QuestStepStatus.Inactive:
-                    // do nothing
-                default:
-                    // do nothing
-                    return;
-            }
-        }
-
         private void AddStepToStepMap(QuestStep step)
         {
-            if(isDebug) Debug.Log($"--- Received request to add step {step.name} with id: {step.StepId} to stepMap.");
+            if (isDebug) Debug.Log($"--- Received request to add step {step.name} with id: {step.StepId} to stepMap.");
             if (!stepMap.ContainsKey(step.StepId))
             {
-                if(isDebug) Debug.Log($"--- Step [{step.StepId}] not found in stepMap, adding it.");
+                if (isDebug) Debug.Log($"--- Step [{step.StepId}] not found in stepMap, adding it.");
                 stepMap.Add(step.StepId, step);
             }
             if (completedSteps.ContainsKey(step.StepId))
             {
-                if(isDebug) Debug.Log($"--- Step [{step.StepId}] already completed removing it from completeSteps so that we can go through it again.");
+                if (isDebug) Debug.Log($"--- Step [{step.StepId}] already completed removing it from completeSteps so that we can go through it again.");
                 completedSteps.Remove(step.StepId);
             }
         }
+        #endregion
+
         #region quest process
         private void Init(string id)
         {
@@ -196,7 +175,26 @@ namespace jeanf.questsystem
 
             LoadDependencies();
         }
-
+        public void UpdateStepStatus(string id, QuestStepStatus status)
+        {
+            switch (status)
+            {
+                case QuestStepStatus.Completed when activeSteps.ContainsKey(id):
+                    // put step in completed list
+                    activeSteps.Remove(id);
+                    completedSteps.Add(id, stepMap[id]);
+                    break;
+                case QuestStepStatus.Active when !activeSteps.ContainsKey(id):
+                    // put step in active list
+                    activeSteps.Add(id, stepMap[id]);
+                    break;
+                case QuestStepStatus.Inactive:
+                // do nothing
+                default:
+                    // do nothing
+                    return;
+            }
+        }
         private void UpdateState()
         {
             if (isDebug) Debug.Log($"Updating State...");
@@ -232,14 +230,12 @@ namespace jeanf.questsystem
                     throw new ArgumentOutOfRangeException();
             }
         }
-
         private void UpdateProgress(string id, float progress)
         {
             if (id != questId) return;
             this.progress = progress;
             if (isDebug) Debug.Log($"questid [{id}] progress = {progress * 100}%");
         }
-
         private void QuestStateChange(Quest quest)
         {
             // only update the quest state if this point has the corresponding quest
@@ -251,6 +247,7 @@ namespace jeanf.questsystem
         }
         #endregion
 
+        #region Complete all Steps
         public void ValidateCurrentlyActiveSteps()
         {
             var currentlyActiveSteps = activeSteps.Keys;
@@ -262,18 +259,15 @@ namespace jeanf.questsystem
                 if(activeSteps.ContainsKey(stepKey)) ValidateStepEvent.Invoke(stepKey);
             }
         }
+        #endregion
 
         #region validation tools
 
-        #if UNITY_EDITOR
+#if UNITY_EDITOR
         public void OnValidate()
         {
             ValidityCheck();
         }
-
-
-        
-
         public void LogActiveSteps()
         {
             Debug.Log($"There is {activeSteps.Count} active steps at the moment.");
@@ -283,7 +277,6 @@ namespace jeanf.questsystem
             }
         }
         #endif
-
         private void ValidityCheck()
         {
             const string searching = "attempting to find";

package/Runtime/Scripts/Core/QuestManager.cs

@@ -11,6 +11,8 @@ namespace jeanf.questsystem
 {
     public class QuestManager : MonoBehaviour, IDebugBehaviour, IValidatable
     {
+        #region variables
+        #region interface variables
         public bool isDebug
         {
             get => _isDebug;
@@ -19,23 +21,26 @@ namespace jeanf.questsystem
         public bool IsValid { get; private set; }
 
         [SerializeField] private bool _isDebug = false;
+        #endregion
 
-        [FormerlySerializedAs("loadQuestState")] [Header("Config")] [SerializeField]
-        private bool loadSavedQuestState = true;
-
+        #region event channels
         [Header("Broadcasting on:")]
         [SerializeField] [Validation("A reference to the questStatusUpdateChannel is required.")] private StringEventChannelSO questStatusUpdateChannel;
         [SerializeField] [Validation("A reference to the questProgress is required.")] private StringFloatEventChannelSO questProgress;
         [SerializeField] [Validation("A reference to the questInitialCheck channel is required.")] private StringEventChannelSO QuestInitialCheck;
-
         [Header("Listening on:")] [SerializeField] [Validation("A reference to the questStatusUpdateRequested is required.")] private StringEventChannelSO questStatusUpdateRequested;
+        #endregion
 
+        #region other variables
+        [FormerlySerializedAs("loadQuestState")] [Header("Config")] [SerializeField]
+        private bool loadSavedQuestState = true;
         private Dictionary<string, Quest> questMap;
-
-
-        // quest start requirements
         private int currentPlayerLevel;
+        #endregion
+        #endregion
 
+        #region Methods
+        #region Standard Unity Methods
         private void Awake()
         {
             questMap = CreateQuestMap();
@@ -45,25 +50,16 @@ namespace jeanf.questsystem
                 CheckIfQuestIsAlreadyLoaded(quest.Key);
             }
         }
-
         private void OnEnable()
         {
             GameEventsManager.instance.questEvents.onStartQuest += StartQuest;
-            GameEventsManager.instance.questEvents.onFinishQuest += FinishQuest;
-            
+            GameEventsManager.instance.questEvents.onFinishQuest += FinishQuest;       
             //GameEventsManager.instance.questEvents.onQuestStepStateChange += QuestStepStateChange;
-
             GameEventsManager.instance.playerEvents.onPlayerLevelChange += PlayerLevelChange;
-
             questStatusUpdateRequested.OnEventRaised += ctx => CheckRequirementsMet(questMap[ctx]);
-
-
         }
-
-
         private void OnDisable() => Unsubscribe();
         private void OnDestroy() => Unsubscribe();
-
         private void Unsubscribe()
         {
             GameEventsManager.instance.questEvents.onStartQuest -= StartQuest;
@@ -74,7 +70,6 @@ namespace jeanf.questsystem
             GameEventsManager.instance.playerEvents.onPlayerLevelChange -= PlayerLevelChange;
 
         }
-
         private void Start()
         {
             foreach (Quest quest in questMap.Values)
@@ -83,25 +78,63 @@ namespace jeanf.questsystem
                 GameEventsManager.instance.questEvents.QuestStateChange(quest);
             }
         }
-
-
-        private void CheckIfQuestIsAlreadyLoaded(string id)
+        private void Update()
         {
-            QuestInitialCheck.RaiseEvent(id);
+            // loop through ALL quests
+            foreach (Quest quest in questMap.Values)
+            {
+                // if we're now meeting the requirements, switch over to the CAN_START state
+                if (quest.state == QuestState.REQUIREMENTS_NOT_MET && CheckRequirementsMet(quest))
+                {
+                    ChangeQuestState(quest.questSO.id, QuestState.CAN_START);
+                }
+            }
         }
-
-        private void ChangeQuestState(string id, QuestState state)
+        private void OnApplicationQuit()
         {
-            Quest quest = GetQuestById(id);
-            quest.state = state;
-            GameEventsManager.instance.questEvents.QuestStateChange(quest);
+            foreach (Quest quest in questMap.Values)
+            {
+                SaveQuest(quest);
+            }
         }
+        #endregion
 
-        private void PlayerLevelChange(int level)
+        #region Quest Checks and getters
+        private Dictionary<string, Quest> CreateQuestMap()
         {
-            currentPlayerLevel = level;
+            // loads all QuestInfoSO Scriptable Objects under the Assets/Resources/Quests folder
+            QuestSO[] allQuests = Resources.LoadAll<QuestSO>("Quests");
+            // Create the quest map
+            Dictionary<string, Quest> questMap = new Dictionary<string, Quest>();
+            foreach (QuestSO questSO in allQuests)
+            {
+                var id = questSO.id;
+                if (questMap.ContainsKey(id))
+                {
+                    Debug.LogWarning("Duplicate ID found when creating quest map: " + questSO.id);
+                }
+                else
+                {
+                    questMap.Add(id, LoadQuest(questSO));
+                }
+                if (isDebug) Debug.Log($"Adding {questSO.name} to the questmap, its id is: {questSO.id}");
+            }
+            return questMap;
+        }
+        private void CheckIfQuestIsAlreadyLoaded(string id)
+        {
+            QuestInitialCheck.RaiseEvent(id);
         }
+        public Quest GetQuestById(string id)
+        {
+            Quest quest = questMap[id];
+            if (quest == null)
+            {
+                Debug.LogError("ID not found in the Quest Map: " + id);
+            }
 
+            return quest;
+        }
         private bool CheckRequirementsMet(Quest quest)
         {
             // check player level requirements
@@ -115,25 +148,20 @@ namespace jeanf.questsystem
                     meetsRequirements = false;
                 }
             }
-            
-            if(isDebug) Debug.Log($"checking requirements for quest: {quest.questSO.name}, [{quest.questSO.id}], meetsRequirements: {meetsRequirements}");
+
+            if (isDebug) Debug.Log($"checking requirements for quest: {quest.questSO.name}, [{quest.questSO.id}], meetsRequirements: {meetsRequirements}");
 
             return meetsRequirements;
         }
-
-        private void Update()
+        private void ChangeQuestState(string id, QuestState state)
         {
-            // loop through ALL quests
-            foreach (Quest quest in questMap.Values)
-            {
-                // if we're now meeting the requirements, switch over to the CAN_START state
-                if (quest.state == QuestState.REQUIREMENTS_NOT_MET && CheckRequirementsMet(quest))
-                {
-                    ChangeQuestState(quest.questSO.id, QuestState.CAN_START);
-                }
-            }
+            Quest quest = GetQuestById(id);
+            quest.state = state;
+            GameEventsManager.instance.questEvents.QuestStateChange(quest);
         }
+        #endregion
 
+        #region main process
         private void StartQuest(string id)
         {
             Quest quest = GetQuestById(id);
@@ -143,7 +171,6 @@ namespace jeanf.questsystem
             quest.messageChannel.RaiseEvent(quest.messageToSendOnInit);
             if(isDebug) Debug.Log($"quest id:{id}  started, a message was attatched to the initialization: {quest.messageToSendOnInit}");
         }
-
         private void UpdateProgress(Quest quest)
         {
             var progress = 0;
@@ -151,7 +178,6 @@ namespace jeanf.questsystem
             if (isDebug) Debug.Log($"[{quest.questSO.id}] progress: {progress * 100}%", this);
             questProgress.RaiseEvent(quest.questSO.id, progress);
         }
-
         private void FinishQuest(string id)
         {
             Quest quest = GetQuestById(id);
@@ -163,58 +189,26 @@ namespace jeanf.questsystem
             SaveQuest(quest);
             if (!quest.sendMessageOnFinish) return;
         }
+        #endregion
 
+        #region rewards and progress
         private void ClaimRewards(Quest quest)
         {
             GameEventsManager.instance.scenarioEvents.ScenarioUnlocked(quest.questSO.unlockedScenario);
         }
 
-        private Dictionary<string, Quest> CreateQuestMap()
-        {
-            // loads all QuestInfoSO Scriptable Objects under the Assets/Resources/Quests folder
-            QuestSO[] allQuests = Resources.LoadAll<QuestSO>("Quests");
-            // Create the quest map
-            Dictionary<string, Quest> questMap = new Dictionary<string, Quest>();
-            foreach (QuestSO questSO in allQuests)
-            {
-                var id = questSO.id;
-                if (questMap.ContainsKey(id))
-                {
-                    Debug.LogWarning("Duplicate ID found when creating quest map: " + questSO.id);
-                }
-                else
-                {
-                    questMap.Add(id, LoadQuest(questSO));
-                }
-                if(isDebug) Debug.Log($"Adding {questSO.name} to the questmap, its id is: {questSO.id}");
-            }
-
-            return questMap;
-        }
-
-        public Quest GetQuestById(string id)
-        {
-            Quest quest = questMap[id];
-            if (quest == null)
-            {
-                Debug.LogError("ID not found in the Quest Map: " + id);
-            }
-
-            return quest;
-        }
-
-        private void OnApplicationQuit()
+        private void PlayerLevelChange(int level)
         {
-            foreach (Quest quest in questMap.Values)
-            {
-                SaveQuest(quest);
-            }
+            currentPlayerLevel = level;
         }
+        #endregion
 
+        #region saving and loading
         private void SaveQuest(Quest quest)
         {
             try
             {
+                //Save; active steps + quest step status for each, completed steps, quest status, progress/playerLevel/?
                 QuestData questData = null;
                 //quest.GetQuestData();
                 // serialize using JsonUtility, but use whatever you want here (like JSON.NET)
@@ -230,7 +224,6 @@ namespace jeanf.questsystem
                 Debug.LogError("Failed to save quest with id " + quest.questSO.id + ": " + e);
             }
         }
-
         private Quest LoadQuest(QuestSO questSO)
         {
             Debug.Log($"attempting to load quest with id: [{questSO.id}]");
@@ -259,6 +252,9 @@ namespace jeanf.questsystem
 
             return quest;
         }
+        #endregion
+
+        #region Validation Tools
         private void ValidityCheck()
         {
             const string searching = "attempting to find";
@@ -334,5 +330,7 @@ namespace jeanf.questsystem
             ValidityCheck();
             #endif
         }
+        #endregion
+        #endregion
     }
 }

package/Runtime/Scripts/Core/QuestStep.cs

@@ -13,15 +13,17 @@ namespace jeanf.questsystem
     [System.Serializable, NodeMenuItem("questSystem/QuestStep"), DefaultExecutionOrder(1)]
     public class QuestStep : MonoBehaviour, IDebugBehaviour
     {
+        #region Ids and status
         [field: Space(10)][field: ReadOnly][SerializeField] string stepId;
         public string StepId { get { return stepId; } }
 
         string questId;
         public string QuestId { get { return questId; } }
+        [field: ReadOnly][SerializeField] public QuestStepStatus stepStatus;
+        #endregion
 
-        
-        [field: ReadOnly] [SerializeField] public QuestStepStatus stepStatus;
 
+        #region timeline 
         [Tooltip("This boolean has to be enabled if the quest step has an intro timeline.")]
         public bool isUsingIntroTimeline = false;
 
@@ -29,7 +31,9 @@ namespace jeanf.questsystem
         [SerializeField] private TimelineTriggerEventChannelSO _timelineTriggerEventChannelSo;
         [DrawIf("isUsingIntroTimeline", true, ComparisonType.Equals, DisablingType.DontDraw)] 
         public PlayableAsset timeline;
+        #endregion
 
+        #region step trigger and completion
         [Header("Quest Step Progression events & Variables")]
         public List<QuestStep> questStepsToTrigger = new List<QuestStep>();
         public delegate void SendNextStepId(string id);
@@ -41,14 +45,18 @@ namespace jeanf.questsystem
         public static StepActive stepActive;
         public delegate void ChildStep(QuestStep step);
         public static ChildStep childStep;
-        
+        #endregion
+
+        #region events
         [Header("Quest Tooltip")]
         [SerializeField] private QuestTooltipSO questTooltipSO;
 
         [Header("Event Channels")]
         [SerializeField] private StringEventChannelSO sendQuestStepTooltip;
         [SerializeField] private StringEventChannelSO stepValidationOverride;
-        
+        #endregion
+
+        #region standard unity methods
         public void OnEnable()
         {
             Subscribe();
@@ -65,13 +73,14 @@ namespace jeanf.questsystem
             if(stepValidationOverride) stepValidationOverride.OnEventRaised += ValidateCurrentStep;
         }
 
-        private void Unsubscribe()
+        protected virtual void Unsubscribe()
         {
             QuestItem.ValidateStepEvent -= ValidateCurrentStep;
             if(stepValidationOverride) stepValidationOverride.OnEventRaised -= ValidateCurrentStep;
         }
+        #endregion
 
-
+        #region step progress
         public void InitializeQuestStep()
         {
             // failsafe to avoid lauching the same step more than once at a time.
@@ -91,6 +100,7 @@ namespace jeanf.questsystem
             {
                 if(isDebug) Debug.Log($"sending trigger to timeline: {timeline.name}, triggerValue: true");
                 _timelineTriggerEventChannelSo.RaiseEvent(timeline, true);
+                
             }
 
             if(isDebug) Debug.Log($"Step with id [{stepId}] has {questStepsToTrigger.Count} childSteps");
@@ -107,7 +117,6 @@ namespace jeanf.questsystem
             FinishQuestStep();
         }
 
-
         public void FinishQuestStep()
         {
             if(isDebug) Debug.Log($" ---- Step with id: {stepId} finished. Changing status to completed", this);
@@ -118,23 +127,28 @@ namespace jeanf.questsystem
                 if(isDebug) Debug.Log($" ---- Step with id: {stepId} finished. Sending tooltip", this);
                 sendQuestStepTooltip.RaiseEvent(string.Empty);
             }
-;
 
-            foreach(QuestStep questStep in questStepsToTrigger)
-            {
-                if(isDebug) Debug.Log($" ---- Step with id: {stepId} finished. Requesting to start next step: {questStep.stepId}", this);
-                sendNextStepId?.Invoke(questStep.stepId);
-            }
+
             if(isDebug) Debug.Log($" ---- Step with id: {stepId} finished. Sending stepCompleted Event (delegate) with argument: {stepId}", this);
             stepCompleted?.Invoke(stepId);
-            if(isDebug) Debug.Log($" ---- Step with id: {stepId} finished. Sending stepActive Event (delegate) with arguments: {stepId}, {stepStatus} ", this);
+            if (isDebug) Debug.Log($" ---- Step with id: {stepId} finished. Sending stepActive Event (delegate) with arguments: {stepId}, {stepStatus} ", this);
             stepActive?.Invoke(stepId, stepStatus);
 
+            foreach (QuestStep questStep in questStepsToTrigger)
+            {
+                if (isDebug) Debug.Log($" ---- Step with id: {stepId} finished. Requesting to start next step: {questStep.stepId}", this);
+
+                //Si questStep.prerequisitesStep are in QuestItem.stepsCompleted
+                sendNextStepId?.Invoke(questStep.stepId);
+            }
+
+
             if(isDebug) Debug.Log($" ---- Step with id: {stepId} finished. Destroying the gameobject with name {this.name}", this);
             Destroy(this.gameObject);
         }
+        #endregion
 
-
+        #region tooltip
         protected void DisplayActiveQuestStep()
         {
             if (questTooltipSO != null)
@@ -142,8 +156,10 @@ namespace jeanf.questsystem
                 sendQuestStepTooltip.RaiseEvent(questTooltipSO.Tooltip);
             }
         }
+        #endregion
 
-        #if UNITY_EDITOR
+        #region validation
+#if UNITY_EDITOR
         private void OnValidate()
         {
             if (stepId == string.Empty || stepId == null) GenerateId();
@@ -155,16 +171,20 @@ namespace jeanf.questsystem
             UnityEditor.EditorUtility.SetDirty(this);
         }
 
-        #endif
-
+#endif
+        #endregion
 
+        #region debug
         public bool isDebug { get => _isDebug; set => _isDebug = value; }
         private bool _isDebug = true;
+        #endregion
 
+        #region Status
         public QuestStepStatus GetStatus()
         {
             return stepStatus;
         }
+        #endregion
     }
 
     public enum QuestStepStatus

package/Runtime/Scripts/GraphEditor/QuestGraphWindow.cs

@@ -0,0 +1,30 @@
+#if UNITY_EDITOR
+using UnityEngine;
+using UnityEditor;
+using GraphProcessor;
+
+public class QuestGraphWindow : BaseGraphWindow
+{
+    [MenuItem("QuestGraph/01_DefaultGraph")]
+    public static BaseGraphWindow Open()
+    {
+        var graphWindow = GetWindow<QuestGraphWindow>();
+
+        graphWindow.Show();
+
+        return graphWindow;
+    }
+
+    protected override void InitializeWindow(BaseGraph graph)
+    {
+        // Set the window title
+        titleContent = new GUIContent("Default Graph");
+
+        // Here you can use the default BaseGraphView or a custom one (see section below)
+        var graphView = new BaseGraphView(this);
+
+        rootView.Add(graphView);
+    }
+    
+}
+#endif

package/Runtime/Scripts/GraphEditor/QuestGraphWindow.cs.meta

@@ -0,0 +1,11 @@
+fileFormatVersion: 2
+guid: 4783b98f3c3c6da49b8d8a1dc5b2bd52
+MonoImporter:
+  externalObjects: {}
+  serializedVersion: 2
+  defaultReferences: []
+  executionOrder: 0
+  icon: {instanceID: 0}
+  userData: 
+  assetBundleName: 
+  assetBundleVariant: 

package/Runtime/Scripts/GraphEditor/QuestStepNode.cs

@@ -0,0 +1,11 @@
+using UnityEngine;
+using GraphProcessor;
+using jeanf.questsystem;
+
+public class QuestStepNode : BaseNode
+{
+    [Input(name = "A")] 
+    private QuestStep StepsRequiredForCompletion;
+    [Output(name = "Out")]
+    private QuestStep StepsTriggeredOnCompletion;
+}

package/Runtime/Scripts/GraphEditor/QuestStepNode.cs.meta

@@ -0,0 +1,11 @@
+fileFormatVersion: 2
+guid: feb5db492d0ac36459e73370531d6f8c
+MonoImporter:
+  externalObjects: {}
+  serializedVersion: 2
+  defaultReferences: []
+  executionOrder: 0
+  icon: {instanceID: 0}
+  userData: 
+  assetBundleName: 
+  assetBundleVariant: 

package/Runtime/Scripts/GraphEditor.meta

@@ -0,0 +1,8 @@
+fileFormatVersion: 2
+guid: 7b22fa752a8ac804aa1053eac644fb47
+folderAsset: yes
+DefaultImporter:
+  externalObjects: {}
+  userData: 
+  assetBundleName: 
+  assetBundleVariant: 

package/Runtime/Scripts/jeanf.QuestSystem.asmdef

@@ -7,7 +7,9 @@
         "GUID:e8958e6939af7314a97769de4be4ce25",
         "GUID:db1b0a3157155ad47996061dbcc07bbb",
         "GUID:ca937d03ee5dd4d699091438dc0f3ae6",
-        "GUID:a432eb0a70c5c4a37ab50d59671586f8"
+        "GUID:a432eb0a70c5c4a37ab50d59671586f8",
+        "GUID:b8e24fd1eb19b4226afebb2810e3c19b",
+        "GUID:002c1bbed08fa44d282ef34fd5edb138"
     ],
     "includePlatforms": [],
     "excludePlatforms": [],

package/package.json

@@ -1,22 +1,20 @@
 {
-  "name":"fr.jeanf.questsystem",
-  "version":"0.0.61",
-  "displayName":"Quest system",
-  "description":"This package uses Scriptable Objects to define quests.",
+  "name": "fr.jeanf.questsystem",
+  "version": "0.0.63",
+  "displayName": "Quest system",
+  "description": "This package uses Scriptable Objects to define quests.",
   "unity": "2021.3",
   "keywords": [
     "jeanf",
     "quest system"
   ],
-  "author":{
-    "name":"Jean-François Robin",
-    "email":"robin.jeanfrancois@gmail.com",
-    "url":"https://jeanfrancoisrobin.art"
+  "author": {
+    "name": "Jean-François Robin",
+    "email": "robin.jeanfrancois@gmail.com",
+    "url": "https://jeanfrancoisrobin.art"
   },
   "dependencies": {
     "fr.jeanf.propertydrawer": "1.1.6"
   },
-  "samples": [ 
-    
-  ]
+  "samples": []
 }