Skip to content

Add sortOrder field to Term and update terms retrieval sorting logic#23

Merged
subsub97 merged 1 commit intomainfrom
orderby-term
Feb 21, 2026
Merged

Add sortOrder field to Term and update terms retrieval sorting logic#23
subsub97 merged 1 commit intomainfrom
orderby-term

Conversation

@jeyongsong
Copy link
Member

This pull request introduces a new sortOrder field to the Term entity and updates the sorting logic in the TermsService to use this field. These changes improve how terms are ordered when retrieved, allowing for more flexible and explicit ordering.

Entity model update:

  • Added a new sortOrder field to the Term entity in Term.kt, which enables explicit ordering of terms.

Service logic update:

  • Updated the sorting logic in TermsService.kt to use the new sortOrder field, replacing the previous sorting by required and code.

@jeyongsong
Copy link
Member Author

이거 득호책임님이 머지하시고 데이터 대응도 해주세용~~ ㅎㅎㅎ

@github-actions
Copy link

Test Results

17 tests   17 ✅  0s ⏱️
 2 suites   0 💤
 2 files     0 ❌

Results for commit ee4fc5c.

Copy link
Collaborator

@subsub97 subsub97 left a comment

Choose a reason for hiding this comment

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

ap ap 가시죠

@subsub97 subsub97 merged commit 8d8c51d into main Feb 21, 2026
2 checks passed
@subsub97 subsub97 deleted the orderby-term branch February 21, 2026 17:35
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