Skip to content

스니펫추가: drag이벤트를 이용한 파일 드래그앤드롭 사용 예제 #32

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

Merged
merged 2 commits into from
Jan 19, 2017

Conversation

devjin0617
Copy link
Member

스니펫 PR요청합니다.
그냥 master에 넣는 것보다 몇명의 동의를 구하고 진행하는 것이 좋을 것 같아 PR에 등록합니다.

@ChangJoo-Park
Copy link
Member

감사합니다
그런데 PR로 해야하면 동의를 해야하는 조건이 필요하겠네요
어떻게 할까요?

@devjin0617
Copy link
Member Author

commit에 대한 리뷰를 진행하여 수정될사항있으면 코멘트달고, 없으면 완료댓글정도만 작성하여 머지진행하면 어떨까요?

@ChangJoo-Park
Copy link
Member

네 좋을거 같아요 fiddle은 작동상에 문제가 일부 있더라도 목적에 맞는 기능을 하고 있다면 머지하는게 좋겠네요
너무 과하게 이것저것 다 되야하면 문제가 될 여지가 있어 보여요

그리고 리뷰가 필요하긴한데 PR 을 거쳐야하는 무게감을 어떻게 최소화할지 고려해야 하겠네요

@ChangJoo-Park ChangJoo-Park merged commit 8cb71d9 into vuejs-kr:master Jan 19, 2017
@devjin0617 devjin0617 deleted the devjin0617-patch-2 branch February 13, 2017 12:56
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.

2 participants