Skip to content

Step6 PR - #152

Merged
javajigi merged 4 commits into
code-squad:zingoworksfrom
vamboo26:step6
Jan 28, 2019
Merged

javajigi merged 4 commits into
code-squad:zingoworksfrom
vamboo26:step6

Conversation

@vamboo26

Copy link
Copy Markdown

포비 안녕하세요!
스텝 6 PR 드립니다.
코드 리팩토링, 기타 미비 된 테스트 코드 구현은 다음 스텝에서 진행 예정입니다.

zingoworks added 4 commits January 23, 2019 12:29
텍스트파일 아닌 JPG, PNG, JPEG 파일 다운로드 가능하도록 구현

@javajigi javajigi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

첨부 파일이 n개인 경우도 고려하면 더 좋을 것 같네요.

private Issue issue;

@OneToOne
private Attachment attachment;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

첨부 파일이 하나가 아니라 n개가 될 수도 있는데 그 부분까지 고려하면 좋을 것 같네요.

@javajigi
javajigi merged commit 28c63e7 into code-squad:zingoworks Jan 28, 2019
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