The Wayback Machine - https://web.archive.org/web/20210902133843/https://github.com/nextcloud/server/issues/28119
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix vue/no-mutating-props issues #28119

Open
skjnldsv opened this issue Jul 22, 2021 · 0 comments
Open

Fix vue/no-mutating-props issues #28119

skjnldsv opened this issue Jul 22, 2021 · 0 comments

Comments

@skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented Jul 22, 2021

We badly started implementing some props mutation.
There are usually two cases:

  1. The props are injected via root component propsData
  2. The props are used inside a vue app

Solution 1

Migrate the propsData (usually initial-states) to the root .vue file away from the .js entry point

Solution 2

Use a sync modifier and change the data flow according to vue documentations. Or rethink the way you're using the data and consider switching to events and move the handling methods one component up.

@skjnldsv skjnldsv mentioned this issue Jul 22, 2021
2 tasks
@skjnldsv skjnldsv added this to Backlog in 📁 Files team via automation Jul 23, 2021
@skjnldsv skjnldsv removed this from Backlog in 📁 Files team Aug 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant