Skip to content

Refactor[COMMON]: MenuHeader 공통화#103

Open
MatchaKim wants to merge 3 commits into
eGovFramework:mainfrom
MatchaKim:refactor(MenuHeaderComponent)
Open

Refactor[COMMON]: MenuHeader 공통화#103
MatchaKim wants to merge 3 commits into
eGovFramework:mainfrom
MatchaKim:refactor(MenuHeaderComponent)

Conversation

@MatchaKim
Copy link
Copy Markdown
Contributor

@MatchaKim MatchaKim commented Sep 30, 2025

수정 사유 Reason for modification

소스를 수정한 사유가 무엇인지 체크해 주세요. Please check the reason you modified the source. ([X] X는 대문자여야 합니다.)

  • 버그수정 Bug fixes
  • 기능개선 Enhancements
  • 기능추가 Adding features
  • 기타 Others

수정된 소스 내용 Modified source

검토자를 위해 수정된 소스 내용을 설명해 주세요. Please describe the modified source for reviewers.

새로운 EgovMenuHeader 컴포넌트를 도입해 페이지 전반의 제목 표시 방식을 통합했습니다.
중복되던 코드들을 이 컴포넌트로 치환하여 일관성 있는 UI를 유지하면서도 중복을 줄여 유지보수성을 개선했습니다.

JUnit 테스트 JUnit tests

테스트를 완료하셨으면 다음 항목에 [대문자X]로 표시해 주세요. When you're done testing, check the following items.

  • JUnit 테스트 JUnit tests
  • 수동 테스트 Manual testing

테스트 브라우저 Test Browser

테스트를 진행한 브라우저를 선택해 주세요. Please select the browser(s) you ran the test on. (다중 선택 가능 you can select multiple) [X] X는 대문자여야 합니다.

  • Chrome
  • Firefox
  • Edge
  • Safari
  • Opera
  • Internet Explorer
  • 기타 Others

테스트 스크린샷 또는 캡처 영상 Test screenshots or captured video

테스트 전과 후의 스크린샷 또는 캡처 영상을 이곳에 첨부해 주세요. Please attach screenshots or video captures of your before and after tests here.

localhost_3000_support_download localhost_3000_support_download (1)

@MatchaKim MatchaKim changed the title refactor : MenuHeader 공통화 Refactor[COMMON]: MenuHeader 공통화 Sep 30, 2025
@eGovFrameSupport
Copy link
Copy Markdown
Contributor

표준프레임워크에 대한 지속적인 참여에 대단히 감사드립니다.
반복되던 페이지 헤더 마크업(top_tit/tit_1/tit_2/tit_7)을 EgovMenuHeader 한 컴포넌트로 통합한 것을 확인했으며,
33개 적용 파일을 모두 검토해 치환이 원본과 동작상 동일함을 확인했습니다.

다만 본 PR은 아래 세 가지 main 변경이 반영되기 이전 시점에서 분기되어, 현재 main과 4개 파일에서 충돌합니다(EgovAdminBoardEdit, EgovAdminMemberEdit, EgovMypageEdit, EgovNoticeDetail).
현행 main 기준 rebase를 요청드리며, 이때 아래 main 변경이 되돌려지지 않도록 충돌 해소에 주의 부탁드립니다.

  1. console 디버그 로그 (커밋 85f50b6에서 페이지 컴포넌트의 활성 console.* 일괄 제거됨)

본 PR 분기에는 console.group/log/groupEnd가 남아 있습니다. 대부분은 자동 병합 시 main의 삭제가 유지되지만, getSubTitle
추가 위치가 파일 끝 console.log("... [End]") / console.groupEnd() 와 인접해 충돌합니다.
이 충돌 영역에서는 getSubTitle 함수만 남기고 console.log/console.groupEnd 라인은 제거해 main과 일치가 필요합니다. (Board/Member/MypageEdit 공통)

  1. React 19 ref 콜백 (커밋 204a3ce)
    React 19에서는 ref 콜백이 값을 반환하면 cleanup 함수로 해석되므로, main이 모든 ref를 블록형으로 전환했습니다.
  • main: ref={(el) => { checkRef.current[i] = el; }}
  • 본 PR(이전 형태): ref={(el) => (checkRef.current[i] = el)}
  1. EgovNoticeDetail 인증 방식 (main에서 getSessionItem → useAuth로 전환)
    main은 const { roles } = useAuth() 컨텍스트 기반으로 바뀌었고 getSessionItem import는 제거됐습니다.
    충돌 해소 시 main의 useAuth 방식을 채택하고, 그 위에 EgovMenuHeader import/치환으로 적용 바랍니다.
    (본문의 세션 기반 코드도 main 기준으로 정리 필요)

rebase가 반영되면 다시 빌드·동작까지 확인하겠습니다.

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