Skip to content

feat: 카탈로그 목록에 표시 순서 규칙 적용 - #30

Merged
yessjun merged 1 commit into
mainfrom
feat/catalog-display-order
Aug 8, 2026
Merged

feat: 카탈로그 목록에 표시 순서 규칙 적용#30
yessjun merged 1 commit into
mainfrom
feat/catalog-display-order

Conversation

@yessjun

@yessjun yessjun commented Aug 8, 2026

Copy link
Copy Markdown
Member

📝 작업내용

  • OS 카탈로그와 사양 프리셋의 조회에 표시 순서 규칙 적용
    • 종전에는 등재 순서라 신청서의 OS 목록이 규칙 없는 나열로 보임
    • 등재 순서는 구현의 부산물이고 고르는 사람에게는 의미가 없음
  • OS 는 계열 오름차순 다음 릴리스 오름차순
    • 릴리스는 문자열이라 사전순으로는 열이 아홉보다 앞에 오므로 숫자열로 비교
    • 그 비교식은 컬럼을 만든 마이그레이션이 주석으로 처방해 둔 것
    • 정렬용 컬럼을 새로 두지 않아 기존 행을 채우는 삽입이 필요 없음
  • 사양 프리셋은 vCPU 와 메모리와 디스크 오름차순
    • 계열 축이 없어 크기가 곧 고르는 축
  • 관리자 목록도 같은 순서로 통일
    • 종전에는 다른 조회를 써서 두 화면의 순서가 달랐음
  • 배치 계산에 쓰는 조회는 결과를 집합으로 접어 순서를 쓰지 않으므로 이름에서 순서 주장을 제거

⭐️ 검증

  • api verify.sh 통과, 체크스타일과 mvn verify 전체 테스트와 의존성 감사와 공개 위생 검사 green
  • 순서를 고정하는 테스트 네 건 추가
    • 열을 아홉보다 먼저 등재해 등재 순과 사전 순 두 오답이 모두 실패하도록 구성
    • 프리셋보다 작고 큰 행을 나중에 등재해 양끝에 놓이는지 확인
    • 관리자 목록이 같은 순서를 따르는지 은퇴 행 포함해 확인
  • 설계 명세와의 교차 대조 통과
  • 생성 명세 변화는 버전 한 줄, 콘솔 타입 재생성분이 동일해 콘솔 변경 없음

💬 리뷰 포인트

  • 계열 간 순서를 알파벳으로 둔 선택, 주력을 앞에 두면 사용자가 볼 수 없는 규칙이 됨
  • 노드 조회를 그대로 둔 경계

Both wizard axes were read in insertion order, which showed the OS list as
Ubuntu 24.04 then 26.04 then 22.04 - a sequence with no rule to find. OS
images now sort by family, then by release compared as a number sequence
(text order puts Rocky's 10 ahead of 9; the dotted-digit check constraint
from V62 makes the cast total), then by id. Presets sort smallest first.

No schema change: the ordering rides on existing columns, so no migration
and no sort column to backfill. The admin lists follow the same order, and
NodeRepository keeps its id order - its callers score and aggregate rather
than display, and reordering would only disturb placement tie-breaks.
@yessjun
yessjun merged commit 58a1dc0 into main Aug 8, 2026
1 check passed
@yessjun
yessjun deleted the feat/catalog-display-order branch August 8, 2026 17:34
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.

1 participant