Skip to content
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

message object schema #159

Open
ho1234c opened this issue Nov 12, 2017 · 4 comments
Open

message object schema #159

ho1234c opened this issue Nov 12, 2017 · 4 comments

Comments

@ho1234c
Copy link
Collaborator

ho1234c commented Nov 12, 2017

지금 우리 코드에서는 api를 통해서 온 message객체를 raw하게 사용중입니다. message 객체안에 프로퍼티가 많은데, 우리가 필요한것만 적절히 가져오고 validation을 위한 메소드등을 추가한 클래스를 만들면 좋을 것 같습니다. 제가 하다가 #158 에서 막혀서요 ㅎㅎ;

@Brew-Brew
Copy link
Collaborator

Brew-Brew commented Nov 12, 2017

현재 멘토님이 메시지의 모든 프로퍼티를 저장하는 클래스만드는것은 만들어주셔서, 저장되는 메시지의 property를 컬럼으로 나눠서 user, 메시지내용등등 이런식으러 나눠 저장하도록 수정하면 될까요?

@ho1234c
Copy link
Collaborator Author

ho1234c commented Nov 12, 2017

넵. 제 생각엔 api에서 던져준거 그대로 사용하는것 보다 스키마에 래핑되서 사용하는게 좋을것 같습니다

@swpark6
Copy link
Collaborator

swpark6 commented Nov 14, 2017

message 타입 마다 필요한 정보가 각기 다른 곳에 있어서 어댑터를 하나 만들어서 통일 시키던지 해야될것 같습니다..

@ho1234c
Copy link
Collaborator Author

ho1234c commented Nov 14, 2017

여러곳에서 사용될 스키마를 정의하는게 생각보다 어려워서 일단 미뤄놓은 상태입니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants