fix(vue-app): do not preserve state when registering client-side dynamic module

[danielroe]

Types of changes

• Bug fix (a non-breaking change which fixes an issue)

Description

This PR changes the default preserveState: true (on client-side store.registerModule) to instead be true only if there is in fact state to preserve, which allows for registering dynamic store modules on the client-side without having to pass preserveState: false explicitly.

HT to @Timkor for the suggestion.

closes #8445
closes #7046

Checklist:

• My change requires a change to the documentation.

• I have added tests to cover my changes (if not applicable, please state why)
• All new and existing tests are passing.

[codecov-io]

Codecov Report

Merging #8486 (848ddaa) into dev (dd7f767) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #8486   +/-   ##
=======================================
  Coverage   67.96%   67.96%           
=======================================
  Files          91       91           
  Lines        3918     3918           
  Branches     1070     1070           
=======================================
  Hits         2663     2663           
  Misses       1016     1016           
  Partials      239      239           

Flag	Coverage Δ
unittests	67.96% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd7f767...848ddaa. Read the comment docs.