gothinkster / vue-realworld-example-app

An exemplary real-world application built with Vue.js, Vuex, axios and different other technologies. This is a good example to discover Vue for beginners.

Home Page:https://vue-vuex-realworld.netlify.com

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Profile Settings inputs v-model current user state

arivictor opened this issue · comments

Fairly new to Vuex and vue in general but I noticed in Settings.vue that the inputs v-model the currentUser state. This means that if I modify the input it is already directly mutating the state (is this a no-no?).

Which leads onto the button that then sends this.currentUser back to the store, even though the store was already modified via v-model.

<input
     class="form-control form-control-lg"
     type="text"
     v-model="currentUser.username"
     placeholder="Your username"
 />

I tested this by simple displaying `{{ currentUser }} and you can see it being directly modified when the input changes.

Again, new to Vuex/Vue, but I just went through a whole heap of guides saying not to directly mutate the state, instead use an Action -> Mutation -> State.