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.
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
관리자 모드 개선 및 https 변환 메뉴 추가, workflow 추가 #239
base: master
Are you sure you want to change the base?
Conversation
pr test3
Workflow 추가 / 메뉴 추가 버튼 변경 / 불필요한 주석 삭제
menu_list_update groupcode 수정
add Security
menu_list에서 post전송
참고로 @sooobee 님의 |
PHPUnit 테스트 코드가
과제를 내 준 사람이라면 좋은 점수를 줄 필요를 느끼지 못할 것 같습니다. |
@kkigomi 님이 제기한 문제 이외에 좀 더 더하자면, .htaccess 파일의 내용을 변경하고 있는데, 해당 파일에 쓰기 권한이 없어서 해당 동작이 실패할 수도 있습니다. 여담으로, http -> https로 리다이렉트 하는 기능을 추가하기 위해 .htaccess 파일을 임의로 변경하는 것은 적절하지 못해보입니다. Apache 환경뿐 아니라 nginx도 환경도 있을테고, 이미 해당 소스 코드가 .htaccess에 추가되어 있는 상태에서 "https 적용하기" 버튼을 클릭하면 중복되는 부분만 계속 무의미하게 추가될 뿐입니다. |
<h1> | ||
http 를 https로 바꾸기 위해서는 SSL인증서가 필요합니다. | ||
</h1> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<h1> | |
http 를 https로 바꾸기 위해서는 SSL인증서가 필요합니다. | |
</h1> | |
<h1>HTTP를 HTTPS로 바꾸기 위해서는 SSL인증서가 필요합니다.</h1> |
@@ -9,7 +9,7 @@ callback function can be used for any purpose. With minor modifications, the | |||
callback function can be used to create CSV logs, post results to databases, | |||
etc. | |||
|
|||
Please review the test.php script for the example. | |||
Please review the script for the example. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please review the script for the example. | |
Please review the script for the example. |
안녕하세요. SIR 입니다. 코드를 남겨주셔서 감사합니다. 앞으로 개발 하는데 참고하겠습니다. |
안녕하세요. 전북대학교 컴퓨터인공지능학부 4학년 공리, 권시은, 박진용, 김수비 입니다.
오픈소스에 기여해야하는 학교 과제로 인해 그누보드를 직접 사용해보고 다음과 같은 부분을 개선하면 사용자에게 더욱 편리할 것 같아 부족한 걸 알지만 최대한 수정을 진행해 보았습니다.
관리자 모드 1:1 문의 설정(qa_config.php) 창에서 확인 버튼을 저장과 목록 버튼으로 분리
관리자 모드 메뉴 설정에서 메뉴 다중 선택 가능하도록 변경
http 보안 강화를 위한 설명 메뉴 추가
CI를 위한 workflow 추가 (push시 설정 php 버전과의 호환성 체크)
감사합니다.