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.

Sign up for GitHub

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

Jump to bottom

#375 게시판 정렬 기준 컬럼 변경 regdate -> list_order #384

Merged
merged 1 commit into from
Mar 5, 2014
Merged

#375 게시판 정렬 기준 컬럼 변경 regdate -> list_order #384

merged 1 commit into from
Mar 5, 2014

Conversation

largeden
Copy link
Contributor

  • 게시판모듈 설정에서 문서번호 정렬(list_order)을 등록일 정렬로 변경(문서번호 표기는 삭제)
  • default 스킨에서 sort_index의 regdate, last_update 값을 list_order, update_order로 변경

@largeden
#375 게시판 정렬 기준 컬럼 변경 regdate -> list_order
dc307cb 
- 게시판모듈 설정에서 문서번호 정렬(list_order)을 등록일 정렬로 변경(문서번호 표기는 삭제)
- default 스킨에서 sort_index의 regdate, last_update 값을 list_order, update_order로 변경
@ghost ghost self-assigned this Jan 28, 2014
@khongchi
Copy link
Contributor

간단하게 결정할 문제는 아닌것 같습니다. 일단 보류합니다.

@largeden
Copy link
Contributor Author

@khongchi
문서번호가 애초에 list_order로 되어있어서 등록일을 list_order로 하면 등록일 자체가 사라집니다. regdate, last_update로 정렬을 하지 않는것은 향후 XE의 성능에도 도움이 되는 부분이니 잘 고려하셔서 수정해주시기 바랍니다.

@akasima akasima added this to the 1.7.5 milestone Feb 28, 2014
akasima added a commit that referenced this pull request Mar 5, 2014
@akasima
Merge pull request #384 from largeden/develop
3e8563a 
#375 게시판 정렬 기준 컬럼 변경 regdate -> list_order
@akasima akasima merged commit 3e8563a into xpressengine:develop Mar 5, 2014
@akasima akasima assigned akasima and unassigned ghost Mar 5, 2014
@ghost ghost added the type/enhancement label Mar 7, 2014
@sejin7940 sejin7940 mentioned this pull request May 18, 2014
Closed
@akasima
Copy link
Member

akasima commented May 26, 2014

#730 이슈에 의해서 list_order 와 regdate 를 다시 분리합니다.
update_order를 사용해서 수정일에 대해 정렬하는 방식은 유지합니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@akasima akasima

Labels
merged-pull-request type/discussion type/enhancement
Projects
None yet
Milestone
1.7.5
Development

Successfully merging this pull request may close these issues.
3 participants
@largeden @khongchi @akasima