-
Notifications
You must be signed in to change notification settings - Fork 122
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
추천/비추천 개선안 #118
Comments
기본으로 코어에 필요한 기능인가? 부터가 중요하다고 봅니다. 나중에 따로 플러그인으로 제작해서 연결하는 방향이 좋을 것 같아요.
|
@perzona420 기능을 서버에서 쓰도록 하고, 최대한 클라이언트에서 빠르게한다면은 몇가지는 가능한 기능들이 많은 편이긴합니다. |
@kaijess 추천/비추천 취소부분은 어느정도 필요한 요소가 있어 보이고, 비회원추천에 대해서는 어떻게 처리하는 방향이 있는지 알려주시겠습니까? |
@perzona420 제안하는거니깐요.. 무조건 넣어달라는게 아닙니다 ^^; |
#140 이슈 진행중 |
@qw5414 👍 |
추천및 비추의 목록을 가져올 예정이라면, {@
$args = new stdClass();
$args->document_srl = $document_srl;
$output = executeQuery('document.getVotedMemberList', $args);
}
<block loop="$output->data as $val">
내용~
</block> |
#140 추천 비추천 취소 기능은 일딴 추가 해뒀습니다. |
|
그 동안 요청 해오신 기능들이 코어에 거의 완성식으로 구현이 잘되어있습니다. 바로 윗댓글의 채크되지 않은 4개의 항목은 사실 스킨에서 구현이 가능합니다. 해당 DB를 잘 조율하여 스킨에서 구현할 수 있는 부분은 따로 코어에서 제공하지 않고 스킨에서 구현하시는편이 좋을 것 같습니다. 단 스킨에서 구현시 필요한 쿼리문이 modules/document/queries 폴더내에 없으면 새로운 이슈로 등록해주시면 감사하겠습니다. 해당 쿼리의 목적과 사용성을 분명하게 남겨서 어떻게 구현될지를 남겨주시면 최대한 반영할 수 있는 쿼리문들은 추가해드릴 수 있도록 하겠습니다 :) |
나머지 항목들은 "화제의 글" 애드온이나 각종 위젯을 사용하여 쉽게 구현할 수 있습니다. 저런 식으로 검색 조건이 들어갈 경우 성능이 매우 안좋아지므로 해당 기능을 구현하는 서드파티 자료는 캐시를 적극 활용하여야 합니다. 애드온 하나가 서버를 다운시킨다는 것이 바로 저런 상황에서 나오는 얘기입니다. 이렇게 심각한 문제가 될 수 있는 응용사례를 코어에서 직접 제공할 경우 사용을 권장한다는 인상을 줄 수 있으므로, 코어에서는 제공하지 말아야 한다고 생각합니다. |
이미 오래 전부터 use-third-party 라벨이 붙어있었던 이슈인데 닫는 것을 잊었네요. |
현재 기본 추천/비추천 시스템은 포인트를 이용할뿐 너무 단순합니다.
좀더 개선하여 옵션을 두어 운영자에게 선택의 기회를 주었으면 합니다.
The text was updated successfully, but these errors were encountered: