Skip to content

feat: 삭제 전 확인창 추가 & chore: 불필요한 코드 제거#127

Merged
eGovFrameSupport merged 3 commits into
eGovFramework:mainfrom
limlimjo:20260711
Jul 15, 2026
Merged

feat: 삭제 전 확인창 추가 & chore: 불필요한 코드 제거#127
eGovFrameSupport merged 3 commits into
eGovFramework:mainfrom
limlimjo:20260711

Conversation

@limlimjo

@limlimjo limlimjo commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

수정 사유 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.

삭제 전 확인창 추가 & 불필요한 코드 제거

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.

[사이트관리 > 일정관리]
사이트관리 일정관리

[사이트관리 > 게시판생성관리]
사이트관리 게시판생성관리

[사이트관리 > 공지사항관리]
사이트관리 공지사항관리

[사이트관리 > 사이트갤러리관리]
사이트관리 사이트갤러리관리

[사이트관리 > 회원관리]
사이트관리 회원관리

[알림마당 > 공지사항]
알림마당 공지사항

[알림마당 > 사이트갤러리]
알림마당 사이트갤러리

[마이페이지]
마이페이지 회원탈퇴

@eGovFrameSupport

Copy link
Copy Markdown
Contributor

표준프레임워크에 대한 지속적인 참여에
대단히 감사드립니다.

검토 결과 병합에 있어 의미 있는 PR이라고 판단됩니다.

다만 병합 전에 적용 범위를 맞춰주시면 좋겠습니다.
확인창이 일부 삭제 경로에는 빠져있어 사용자 입장에서 "어떤 삭제는 물어보고, 어떤 삭제는 즉시 실행되는" 혼재가 생깁니다.

  • src/pages/mypage/EgovMypageEdit.jsx — 회원탈퇴(deleteMember)
  • src/pages/inform/gallery/EgovGalleryDetail.jsx — 사용자 갤러리 게시글 삭제: 관리자용
    EgovAdminGalleryDetail.jsx에는 확인창이 추가됐지만, 동일 기능의 사용자 화면에는 빠져 있어 짝이 맞지 않습니다.
  • src/pages/admin/schedule/EgovAdminScheduleDetail.jsx — 일정 삭제(onClickDeleteSchedule): 동일하게 확인창이
    없습니다.

위 3개 경로(최소한 회원탈퇴)까지 확인창을 확장해 모든 삭제 동작에 일관되게 적용되도록 보완해 주시면,
이 PR의 취지가 온전히 살아날 것 같습니다. 감사합니다.

@limlimjo

Copy link
Copy Markdown
Contributor Author

검토 의견 감사합니다.
3개 경로 추가 작업하여 올려놓았습니다.

@eGovFrameSupport

Copy link
Copy Markdown
Contributor

표준프레임워크에 대한 지속적인 참여에
대단히 감사드립니다.

@eGovFrameSupport
eGovFrameSupport merged commit b000063 into eGovFramework:main Jul 15, 2026
1 check passed
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