Skip to content

Mobile communication module#1493

Merged
3 commits merged into
xpressengine:developfrom
misol:mobile-message-module
Jul 6, 2015
Merged

Mobile communication module#1493
3 commits merged into
xpressengine:developfrom
misol:mobile-message-module

Conversation

@misol

@misol misol commented May 24, 2015

Copy link
Copy Markdown
Contributor

모바일 환경에서도 쪽지를 사용할 수 있었습니다.
애드온으로 모듈 설정을 하도록 되어 있었는데, 트리거가 도입되기 이전에 개발되었던 것 같습니다.

직관적인 설정이 가능하도록, 모듈의 설정은 모듈로 이전하고, 커뮤니케이션 애드온은 새 쪽지를 알리는 본연의 기능만 남겼습니다.
커뮤니케이션 애드온이 켜져있지 않은 상황(알림센터 사용 등)에도 쪽지를 자유롭게 사용할 수 있게 설정을 할 수 있습니다.

이 PR 이후에 회원 모듈의 모바일 지원을 향상 시키면 XE의 모바일 지원 수준이 한 단계 상승하게 됩니다.
@ngleader, @bnu 님이 빠르게 확인하시고, XE Core에 꼭 반영되길 바랍니다!

수정 내용

  • 커뮤니케이션 애드온의 회원 메뉴 설정 기능을 모듈의 트리거로 처리하여 조금 더 직관적으로 설정하게 됨.
    • 기본값은 모듈이 동작 하도록 함.
  • 모듈을 켜고 끌 수 있도록 함.
  • 모바일에서 친구 추가를 할 수 있게 함.
  • 스타일 아주 조금 다듬음.
  • 친구 추가시 룰셋을 통해 입력되는 값을 검사하도록 수정함(보안 패치일 수 있음)

misol added 3 commits May 24, 2015 13:31
쪽지 모듈의 모바일 지원을 향상시켰습니다.
이 PR 이후에 회원 모듈의 모바일 지원을 향상 시키면 XE의 모바일 지원 수준이 한 단계 상승하게 됩니다.
꼭 반영되길 바랍니다!

## 수정 내용

* 커뮤니케이션 애드온의 회원 메뉴 설정 기능을 모듈의 트리거로 처리하여 조금 더 직관적으로 설정하게 됨.
* 기본값은 모듈이 동작 하도록 함.
* 모듈을 켜고 끌 수 있도록 함.
* 모바일에서 친구 추가를 할 수 있게 함.
* 스타일 아주 조금 다듬음.
모듈이 비활성화 되어 있을때는 트리거가 없는게 정상 이므로, 업데이트 메시지를 표시하지 않습니다.

주석이 반대로 적힌 부분이 있어서 수정합니다.
@ghost ghost self-assigned this May 28, 2015
@ghost ghost modified the milestones: 1.8.4, next-2015-32 Jul 6, 2015
ghost pushed a commit that referenced this pull request Jul 6, 2015
@ghost ghost merged commit be1f2ef into xpressengine:develop Jul 6, 2015
@ghost ghost mentioned this pull request Jul 6, 2015
@ghost ghost removed the merged-pull-request label Jul 6, 2015
@ghost

ghost commented Jul 6, 2015

Copy link
Copy Markdown

이 PR은 32주차 배포 시 포함하겠습니다.

@ghost ghost modified the milestones: next-2015-37, next-2015-32_ver.1.8.8 Aug 18, 2015
@misol

misol commented Aug 18, 2015

Copy link
Copy Markdown
Contributor Author

@bnu 이거 Revert 되었는데.. 다시 포함 안되고 1.8.8. 배포된 것 같아요.

@ghost ghost modified the milestones: next-2015-37, next-2015-41 Sep 21, 2015
@ghost ghost modified the milestones: next-2015-46, next-2015-41 Oct 16, 2015
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant