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

Feature/use computed #31

Merged
merged 2 commits into from
Sep 9, 2021
Merged

Feature/use computed #31

merged 2 commits into from
Sep 9, 2021

Conversation

ktseo41
Copy link
Contributor

@ktseo41 ktseo41 commented Sep 8, 2021

사실 주요한 이유는 v-if, key를 동시 사용한 부분이라고 하는 것 같은데..,
다른 원인인 것 같아 computed를 대신 사용한 코드랑 얽혀서버려서 같이 PR을 날립니다

:key="chartInfos[index].id":key="chartInfos[index].id + chartInfos[index].options.colors[0]" 으로 변경도 해봤었는데 이 부분은 이유가 아닌 것 같아 제외하고 PR 날립니다
반영한 후 이상없이 잘 되기를 !

@ktseo41 ktseo41 merged commit 288d1a3 into develop Sep 9, 2021
@ktseo41 ktseo41 deleted the feature/use-computed branch September 9, 2021 02:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant