Skip to content
This repository has been archived by the owner on Oct 24, 2019. It is now read-only.

destroyed 상태 실행 함수 내 에러 #8

Open
socketbear opened this issue Jul 5, 2019 · 3 comments
Open

destroyed 상태 실행 함수 내 에러 #8

socketbear opened this issue Jul 5, 2019 · 3 comments
Labels
bug Something isn't working

Comments

@socketbear
Copy link

Version

2.0.0

Test Environment

tui.grid 4.0.3

Current Behavior

image
Tui grid를 toast-ui.vue-grid를 통해 사용하고 있는 중입니다. 문제는 tui.grid 4.0.3 내에 포로토 타입 함수 중 destroy 함수가 사라졌더군요. 이전에 사용하던 tui.grid 3.8.0에서는 존재하였습니다.

Expected Behavior

tui grid 내 destory 함수를 대체하는 기능이 있는 지요?
vue wrapper의 변경이 필요할 것이라 보입니다.

여담으로, 기존 기능 중 data와 colums를 watch에서 인지하여 재 변경 했는데, 2.0 버전에서는 사라졌더군요. 혹시 의도하신 건지? 의도 하신 거면 tui grid 사용 하는 부분에서 직접 resetData를 해야하는지 알고 싶습니다.

참고,

  • tui grid 3.8.0 프로토 타입
    image

  • tui grid 4.0.3 프로토 타입
    image

@yongseokleecom
Copy link

관련이슈인데 Grid.vue 에서 destroyed 오류가 발생합니다.

image

@jung-han
Copy link
Member

@socketbear @yongseokleecom 2.0.1 버전에서 수정될 예정입니다.

@jung-han jung-han added the bug Something isn't working label Aug 21, 2019
@js87zz
Copy link
Contributor

js87zz commented Sep 2, 2019

@socketbear
v2.0.1 버전에서 내부적으로 destroy API를 호출하는 것으로 수정되었으니, 별도로 destroy API를 호출하실 필요는 없을 것 같습니다.
watch 를 이용하여 업데이트 되는 것은 의도적으로 제거한 사항입니다. (grid 내부적으로 반응형 데이터를 만들어 사용하기 때문에, Vue에서 observer data로 변경될 경우 사이드 이펙트 및 성능 저하가 발생하여 제거하였습니다.)
아래 예시처럼 ref로 접근하여 메서드를 호출할 것을 권장드립니다.

this.refs.grid.invoke('resetData', data)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants