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

#94 apply flexible display limit #120

Merged
merged 3 commits into from
Jun 15, 2016
Merged

Conversation

yongbin
Copy link
Contributor

@yongbin yongbin commented Jun 15, 2016

#94 에서 이야기했던 부분입니다. 테스트환경이 완전하지 않기때문에 확인 부탁드립니다.

의류준비 select 화면에서 보여줄 최대항목을 설정하는 값이며 기본값은
10입니다.
설정된 $hard_limit이 있다면 추천의류는 $hard_limit - 최근의류수만큼을
보여주며, 없다면 기존과 동일하게 5개만 보여줍니다.
@aanoaa
Copy link
Contributor

aanoaa commented Jun 15, 2016

테스트 환경을 마련하는데 어떤 어려움이 있나요?

@yongbin
Copy link
Contributor Author

yongbin commented Jun 15, 2016

제가 잘 모른다는 어려움이 있습니다

@aanoaa
Copy link
Contributor

aanoaa commented Jun 15, 2016

README 읽어보시고 테스트환경을 마련해보시고 궁금한 점은 남겨주세요.

@yongbin
Copy link
Contributor Author

yongbin commented Jun 15, 2016

넵 지금 해보고 있는중입니다. 궁금한점 있으면 말씀드리겠습니다.

@yongbin
Copy link
Contributor Author

yongbin commented Jun 15, 2016

기존기록 2건 대여 추천 결과 6건

screen_shot_2016-06-15_at_6_13_43_pm

기존기록 0건 대여 추천 10건

screen_shot_2016-06-15_at_6_16_49_pm

기존기록 7건 대여추천 3건

screen_shot_2016-06-15_at_6_16_58_pm

@yongbin
Copy link
Contributor Author

yongbin commented Jun 15, 2016

오타는 a0d1504 에서 수정해주셨고 제가 의도한대로 동작합니다. 머지하겠습니다.

@yongbin yongbin closed this Jun 15, 2016
@yongbin yongbin reopened this Jun 15, 2016
@yongbin yongbin merged commit 557a22a into develop Jun 15, 2016
@yongbin yongbin deleted the #94-apply-flexible-display-limit branch June 15, 2016 09:25
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