Mobile communication module#1493
Merged
3 commits merged intoJul 6, 2015
Merged
Conversation
쪽지 모듈의 모바일 지원을 향상시켰습니다. 이 PR 이후에 회원 모듈의 모바일 지원을 향상 시키면 XE의 모바일 지원 수준이 한 단계 상승하게 됩니다. 꼭 반영되길 바랍니다! ## 수정 내용 * 커뮤니케이션 애드온의 회원 메뉴 설정 기능을 모듈의 트리거로 처리하여 조금 더 직관적으로 설정하게 됨. * 기본값은 모듈이 동작 하도록 함. * 모듈을 켜고 끌 수 있도록 함. * 모바일에서 친구 추가를 할 수 있게 함. * 스타일 아주 조금 다듬음.
모듈이 비활성화 되어 있을때는 트리거가 없는게 정상 이므로, 업데이트 메시지를 표시하지 않습니다. 주석이 반대로 적힌 부분이 있어서 수정합니다.
|
이 PR은 32주차 배포 시 포함하겠습니다. |
Contributor
Author
|
@bnu 이거 Revert 되었는데.. 다시 포함 안되고 1.8.8. 배포된 것 같아요. |
This pull request was closed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
모바일 환경에서도 쪽지를 사용할 수 있었습니다.
애드온으로 모듈 설정을 하도록 되어 있었는데, 트리거가 도입되기 이전에 개발되었던 것 같습니다.
직관적인 설정이 가능하도록, 모듈의 설정은 모듈로 이전하고, 커뮤니케이션 애드온은 새 쪽지를 알리는 본연의 기능만 남겼습니다.
커뮤니케이션 애드온이 켜져있지 않은 상황(알림센터 사용 등)에도 쪽지를 자유롭게 사용할 수 있게 설정을 할 수 있습니다.
이 PR 이후에 회원 모듈의 모바일 지원을 향상 시키면 XE의 모바일 지원 수준이 한 단계 상승하게 됩니다.
@ngleader, @bnu 님이 빠르게 확인하시고, XE Core에 꼭 반영되길 바랍니다!
수정 내용