devapp - Contributing Nextcloud Open Source

  기여를 하기위해 살펴보던 중 CONTRIBUTING.md에서 오타를 발견하여 수정하였다. 이에 대해 pr을 날리고 merge를 받아 contributor가 되었다 . ! 😃

  빠르게 앱 전환을 할 시에 패드워드 창이 나타나지 않아 이를 개선해야 할 점이라고 생각하였다.  멘토님께서 이에 대한 원인을 분석하여 알려주셨는데, 이는 비밀번호 창이 뜨는 조건이 5초 이상 벗어 났을 때로 되어있기 때문이라고 하셨다.  내가 평소에 사용했던 이와 같이 앱 암호가 있는 앱에서는 바로 나타나는 것을 확인할 수 있어서, 이는 특히 보안면에 있어서도 개선되어야 할 점이라고 생각하였다. 따라서 이슈로 enhancement 라벨을 붙여 , 해당 문제에 대한 화면 녹화를 함께 첨부하여 제안하였다.  헌데, 이는 5초라는 짧은 시간이라서 많은 불편이 없을 거라는 답변을 받고 closed 되었다. 이를 통해 비밀번호 요청 조건이 어떠한 이유로 설정된 것인지 알 수 있었다. !

  멘토님의 조언과 함께 이슈를 제안하는 방식으로 오픈 소스에 기여해야겠다고 생각했다. 이 이슈에 대해선 처음에는 이슈를 제안하고, 이에 대해 해결하는 코드 기여까지 하고 싶었다. 따라서, 토스트를 띄우는 방식으로 이를 해결하려 하였고, 멘토님께서 개선하면 좋을 점에 대해서 피드백 해주셨다. 그렇게 다음과 같은 방식으로 이를 완성하였다. 그래서 enhancement를 통해 나의 solution과 함께 이를 pr을 날려도 되는지 제안하였다.   후 approved를 받으면 pr을 날릴 생각이었다. 헌데, 이는 알고보니 키패드에 의해 가려졌던 것이었다. 유나님의 댓글 덕분에 알 수 있었다. ! 그래서 코드를 수정할 때도 이를 처리하는 것으로 보이는 스낵바가 있는데  어디서 뜨는지 안보여서 몰랐었는데 이에 대한 궁금증이 해결되어 조금 더 코드를 다듬었다. 따라서, 이 문제는 enhancement가 아닌 bug 라벨을 받게 되었고 또한 approved 되었다. 하지만 이 분께서 직접 바로 처리하셔서 아쉽게도 코드기여는 하지 못했다,, 😂 그래도 앞으로 이와 같이 이슈를 제안하는 방식으로 오픈소스에 기여를 계속해 나갈 생각이다.

민지님 이슈를 확인하면서 앱에 대해 불편함 점을 발견했다. passcode 화면일 때, 뒤로가기 버튼을 눌러도 반응이 없다. 버그는 아니었고 뒤로가기를 막아놓은 것 같았다. passcode 설정을 안했을 때 앱을 실행시키고 뒤로가기 버튼을 누르면 앱이 닫힌다. passcode 화면에서도 앱이 닫혀야하지 않을까,,?  개발자가 아닌 사용자 입장에서 생각했을 때 뒤로가기 버튼을 눌렀는데 아무런 반응이 없으면 당황할 것 같다. passcode 화면에서 뒤로가기를 눌렀을 때 앱이 닫히던가 홈버튼을 누르라는 메세지라도 띄워주면 좋을 것 같다고 생각하여 이슈로 등록하였다. #7343 멤버 개발자가 passcode을 설정할 때 생기는 이슈로 오해한 것 같다. 그리고 버그라고 생각했는 지,,, 레이블을 bug로 수정하였다. 그리고 스크린샷을 원했는데 해당 이슈를 사진으로 찍기 어려워서 영상으로 녹화하여 링크를 걸었다.  참고 영상   (이슈는 0s-13s에 확인할 수 있다. 42s-46s는 앱에서 뒤로가기를 눌렀을 때 앱이 닫히는 장면을 녹화한 것이다.) Android version: 5 Device model: Samsung Galaxy S9 Nextcloud app version: 3.15.0 Alpha 1 una.kwon Github : https://github.com/Kuuuna98 Nextcloud/android Github : https://github.com/nextcloud/android

 현재 앱에서 틀린 passcode을 입력했을 경우 오류메세지가 보이지 않으므로 메세지를 보이도록 하자는 내용으로 민지님이 이슈 #7331 를 등록했다.  애뮬레이터로 이슈를 확인했는데 passcode이 틀리면 화면 제일 밑에 오류메세지가 출력된다. 애뮬레이터는 키보드로 번호를 입력할 수 있어서 확인 가능했다. 아마 민지님은 스마트폰 키패드에 가려져서 메세지를 못 봤던 것 같다. 오류 메세지를 새롭게 만드는 것이 아닌 키패드 위로 올려야하는 이슈로 수정해야 할 것 같아 해당 내용을 이슈에 추가했다. 멤버 개발자가 approved 레이블로 변경했는데,, pr exists 레이블도 같이 추가하였다. 민지님이 코드 기여를 할 수 있는 좋은 기회였는데,, 너무 아쉬웠다.😭 Android version: 5 Device model: Samsung Galaxy S9 Nextcloud app version: 3.13.1 una.kwon Github : https://github.com/Kuuuna98 Nextcloud/android Github : https://github.com/nextcloud/android

 [20.11.18] 5번째 회의 다섯 번째 회의를 진행하였다.  금일은 각자 선택한 이슈들에 대한 기여 활동 진행 상황을 간단하게 서로 공유하는 시간을 가졌다. 각자 상황은 조금씩 달랐다. 방향을 잡고 이슈를 분석해 질문과 답변으로 정보를 얻는 분들도 계셨지만 이슈를 바꾸는 분도 계셨고 내 경우에는 선택한 이슈가 관리자로 인해 먼저 fix되어 closed되었다. 기여 활동 공유를 통해 잘 진행하시는 분들의 팁을 많이 알게된 시간이었다. 금일 공유된 정보들도 나중에 참고가 필요하면 활용할 수 있도록 각자 블로그에 게시하기로 하였다. 꼼꼼히 살펴봐야겠다..

검색 이슈를 등록한 후 내가 코드로 구현할 수 있는 다른 이슈들이 있는 지 확인하던 중 #7323 이슈를 읽게 되었다.  이 이슈는 버그 이슈였다. 버그 이슈 형식에는 재현 단계를 적는 곳이 있다. 그래서 버그 이슈를 읽을 때는 애뮬레이터나 실제 휴대폰을 사용하여 재현을 해보며 나도 발생하는 오류인지 먼저 확인을 한다. 이슈는 Nextcloud 앱에서 동영상을 실행시면 하단에 seekbar가 나오지 않는다는 내용이었다.  영상 실행시 초기화면에 보이지 않는 seekbar 그리고 동영상 실행중에 영상을 클릭하면 전체보기가 되는 것인지 화면이 커지면서 seekbar가 표시가 된다.  영상 화면을 클릭하면 보이는 seekbar 해당 이슈를 확인했을 때는 멤버 개발자가 스크린샷을 원했다. 내가 같은 오류가 발생해서 스크린샷을 찍어 이슈 내용에 추가하였다.  그리고 같은 개발자가  #7324  이슈를 올렸는 데, 음악을 실행할 때 seekbar가 보이지 않는다는 내용이었다. 멤버 개발자가 #7323과 중복되는 이슈가 아닌가 물어봤는데 해당 개발자는 다른 이슈라고 이야기하였다. 나는 멤버 개발자와 생각이 동일하게 해당이슈는 중복이라고 생각해서 그렇게 생각하는 이유, 스크린샷과 함께 이슈내용에 추가하였다. 이 이슈는 멤버 개발자가 중복이슈로 닫았다. #7323과 #7324 중복이라는 의견 #7323 이슈 닫힘 영상과 음악 모두 실행 초기화면에서는 seekbar가 보이지않는다. 영상은 화면을 클릭하면 전체보기가 되면서 seekbar가 보이지만 음악은 화면이 없기 때문에 클릭이 되지않는다. 멤버 개발자가 미디어를 재생하면 seekbar가 보이도록 코드를 구현하고 .apk 파일을 올려 테스트를 요청하였다. apk파일을 다운받아서 실행해보았고 잘 실행된다고 알려주었다. 👍👍  코드 수정 후 영상 실행 초기화면에서 보이는 seekbar 수빈님이 테스트를 했는데 버그가 발생했다고 올라왔다. 버그 이슈를 확인해...

💚이전글 읽기💚 검색이슈 pt.1:  https://cnu-devapp.blogspot.com/2020/11/nextcloudandroid-pt1.html 검색이슈 pt.2:  https://cnu-devapp.blogspot.com/2020/11/nextcloudandroid-pt1.html 검색 이슈가 앱과 웹에 발생하는 것을 보면 서버에서 해결해야 할 것 같다고 생각했는데, 오늘 멤버 개발자가 이슈를 Nextcloud/android에서 Nextcloud/server로 옮겼다.  #24169 저번에 서버 구축을 하다가 막혀서 중단했는데,, 다시 시작해야겠다. 우선 서버 구축을 하면서 Nextcloud/server 멤버 개발자 의견을 들어봐야할 것 같다. ..코멘트가 아예 안달린다...😭😭 서버에서는 관심없는 이슈일까... una.kwon Github : https://github.com/Kuuuna98 Nextcloud/android Github : https://github.com/nextcloud/android Nextcloud/server Github : https://github.com/nextcloud/server

은경님이 Darkmode issue가 closed된 것 같다고 알려주셨다. 바로 확인해보았다.. 관리자님이 먼저 수정해오셨나보다.. (#6680) 예상보다 빠르게 closed 되었고 fix PR이 merged 되어있었다.. 커멘트라도 달아놓고 진행할걸하는 생각이 많이 들었다😂 번역으로 많이 기여해야겠다...

브라우저에서 테스트를 할 때, 현재 내가 사용하는 서버가 NC20버전이 아닌 것 같아 데모 버전을 사용하였다. 데모버전은 임시 계정을 사용하여 실행해볼 수 있는데 한 계정당 60분동안 사용이 가능하다. 데모버전 사용을 위해 사이트에 접속한다. https://try.nextcloud.com/ 왼쪽 상자에 있는 'Take me there!' 버튼을 누른다. 임시 생성된 계정과 함께 아래의 화면이 보이게 된다. 암호란에 'demo'를 입력하고 로그인을 한다. 60분동안 데모로 NC20버전을 테스트해볼 수 있다.😀 데모 아이디와 비밀번호를 가지고 앱에서 로그인도 가능하다!  앱 NC20버전을 사용해보고 싶다면 데모를 이용하는 방법도 있다.