Don't use beforeDestroy to affect tracking and progress logging logic

[indirectlylit]

Vue lifecycle hooks - particularly beforeDestroy - should not be used to trigger application logic.

The vue component lifecycle hooks are tied to the DOM lifecycle, not the application lifecycle. beforeDestroy should only be used for low-level cleanup operations like removing DOM event listeners, clearing timers, and cleaning up callbacks.

Application logic that might currently be triggered by beforeDestroy should instead be triggered by whatever vuex action or state change actually caused the component to be destroyed.

---

refs:

• Merge pageState rather than overwrite in learn #2034 (comment), resulting change
• Mastery Mismatch Fix #2354
• Fix logging race condition in assessment-wrapper #4018

---

Some specific things to look for:

• this.$emit('stopTracking');
• this.saveAttemptLogMasterLog(false);
• this.stopTracking();

[rtibbles]

Won't fix.