devapp - Contributing Nextcloud Open Source

   💚이전글 읽기💚 이슈 등록 (Issue #7343):  https://cnu-devapp.blogspot.com/2020/11/nextcloudandroid-pin-issue-7343.html PR 날리기:  https://cnu-devapp.blogspot.com/2020/11/pin-pr-issue-7372.html #7372  PR을 날린 후 멤버 개발자 AndyScherzinger가 to review 레이블을 추가해줬다. 1 2 3 4 5 6 7 8 9 10 11 12 13 14 @Override public   boolean  onKeyDown( int  keyCode, KeyEvent event){      if  (keyCode  = =  KeyEvent.KEYCODE_BACK  & &  event.getRepeatCount() = =   0 ){          if (ACTION_CHECK. equals (getIntent().getAction())){             moveTaskToBack( true );             finishAndRemoveTask();         } else   if  (ACTION_REQUEST_WITH_RESULT. equals (getIntent().getAction())  | |   ...

[20.11.24] 6번째 회의 이번 회의에서는 블로그 포스팅을 통하여 활동을 공유 하는 시간을 가졌다. devapp 팀 블로그에 포스팅한 활동들을 서로 살펴보며, 각자의 활동 내용을 공유하였다. 하나씩 살펴보며 서로에게 반응을 해주고, 도움이 될 수 있는 조언도 해주는 등 다양한 피드백을 주고 받을 수 있는 시간이었다. 💪👏 확실히 팀블로그에 포스팅을 하여서 활동들을 살펴보다보니  보다 더 자세히 각자의 활동을 함께 살펴볼 수 있고, 댓글을 통해 반응하고 조언을 주고 받을 수 있기 때문에 협업에 많은 도움이 되고 편리한 것 같다. ! 😃👍🙆  

  💚이전글 읽기💚 PR 날리기:  https://cnu-devapp.blogspot.com/2020/11/nextcloudandroid-passcode-pr-issue-7370.html l passcode 레이아웃 정렬을 수정한 후 날린 PR #7370 에 멤버 개발자 AndyScherzinger가 to review 레이블을 달아줬다.😍 그리고 다른 멤버 개발자 tobiasKaminsky에게 리뷰를 요청했다. tobiasKaminsky는 중앙 정렬이 아닌 왼쪽 정렬을 한 것이 더 일반적이라는 코멘트와 함께 AndyScherzinger와 nextcloud/designers를 태그 걸어 어떻게 생각하는지 물어봤다. 나는 중앙 정렬이 더 예쁘다고 생각했는 데,, 디자인은 너무 어렵다.😂😂 AndyScherzinger는 타이틀은 왼쪽정렬, PIN은 중앙정렬, 취소버튼은 오른쪽 정렬이어야 한다고 코멘트를 남겼다. 그리고 현재와 같은 디자인이 아닌 어두운 배경에 둥근 네모 상자에 이들을 표시하는 게 좋다고 했다. 현재는 하얀 배경에 요소들이 표시되어있는데, AndyScherzinger의 추가 의견은 괜찮다고 생각한다. Jancborchardt도 코멘트를 남겨줬다. 사실 Nextcloud/android에서 처음 보는 개발자라서 우리처럼 이슈 관리하고 있는 개발자인가 했는데 멤버 개발자표시가 있었다!! 프로필을 보니 오픈소스 디자이너였다! 아까 태그걸린 nextcloud/designers와 관련있는 사람인가보다. 디자이너...멋있다..👍🏻 내 PR을 nice enhancement라고 해줬다. 감동... 😭😭😭 Jancborchardt는 코멘트를 정말 깔끔하게 정리했다. 우선 전제화면이아닌 다이얼로그처럼 보이게 하자는 AndyScherzinger 의견에 동의했다. 취소 버튼은 이곳에서 주가 아니게 때문에 기본으로 스타일을 지정해서는 안되고 중앙에 정렬해야 한다고 한다.  Jancborchardt는 디자인을 추가로 수정하는 ...

 나도 contributor가 되고 싶어서 열심히 오타를 찾아보았다.. (정말 안보였다........) 그런데!!! 열심히 보다보니 세상에는 존재하지 않는 단어가 있었다. [CONTRIBUTING.md의 수정 부분.. 현재는 merged되어 수정 되어있다 :)] 'bufixing'이라는 단어였는데 아무리 봐도 이런 단어는 없었고.. 그러면 buffing인가.. bug fixing 인가.. 번역기에 두 단어 다 넣어보고 번역해봤을 때 bug fixing이 문맥에 좀 더 맞는 것 같아서 bug fixing으로 PR을 날리려하였다. 근데 또 문제가 생겼다. 예전에 fork 해놨던 repository가 업데이트를 안해놓은 상태였었다. (다른 팀원분들이 수정한 부분이 내 repository에는 그대로인 것을 보고 알아챘다.) 구글링을 통해 repository 업데이트 방법을 확인하고 업데이트를 먼저 진행했다!(https://velog.io/@k904808/Fork-한-Repository-업데이트-하기) 그런 다음 첫 PR을 진행해보았다. PR올린지 20분만에 관리자님이 merge해주셨다!!! 드디어 contributor가 되었다 ㅠㅠ

Nextcloud에 번역으로 기여를 하기 위해서는 어떻게 해야할까? github Nextcloud안에 CONTRIBUTING.md의 Translations를 확인해보면 친절하게도 번역으로 기여하는 방법이 자세하게 나와있다. 번역은 다음과 같다  'Nextcloud는 Transifex 라는 웹 기반의 번역 플랫폼을 사용하여 번역을 하며 Transife x에 가입하고 Nextcloud 번역 팀에 참가하여 번역을 진행하면 된다. 또한 모든 번역은 자동으로 repository에 push되어 PR이 따로 필요없다. 다만 바뀐 번역에 대한 PR을 제출할 때는 values/strings.xml의 변경사항만 제출하고 번역 파일은 변경하지 말아달라'고 적혀있다. Transifex에 가입하여 Nextcloud 팀에 참여하면 다음과 같이 다양한 언어의 번역 진행도를 확인할 수 있다. 한국어는 현재 60.23% 번역되어 있으며 그중에 검증된 번역은 29.55% 이다. 번역을 위해 Korean(ko) 를 눌러보면 다음과 같이 여러 분야에서 번역이 어느 정도 되었는지 확인할 수 있다. android-deck에 들어가보자. 현재 번역된 부분과 번역되지 않은 부분, 그리고 해당 번역이 검증이 되었는지 확인할 수 있다. 예전에 번역한 부분이라 캡쳐는 못했지만 그 당시에 unreviewed strings 를 확인하다가 [예끼치 않은] 이라는 잘못된 번역을 [예기치 않은]으로 수정한 적이 있다. 이런 부분들도 체크하면서 진행해야한다.

  동영상을 바로 촬영해서 올리는 기능 추가를 제안하였다. "+" 버튼을 누르면 카메라로부터 바로 업로드가 가능한데,  사진만 가능하고 동영상이 안되기 때문이다 ,,, 전에 사용했던 클라우드 앱에서는 되는 기능이었기 때문에  개선되면 좋을 것이라 생각하여 제안하였다. 개발자님의 반응을 얻진 못했지만 closed 하지 않으시고 남겨 놓으셨기 때문에,,  가능하다면 이에 대한 코드를 구현하고 pr을 날려도 되는지 묻고 날려보고자 한다,,

공백의 이름으로 파일을 Rename 할 때 발생하는 문제에 대해 이슈를 작성했다. Nextcloud 애플리케이션의 메인 화면에서 파일을 선택하고 공백으로 이름을 바꾸고, 새로 고침을 하면 메인화면에서 바뀐이름의 파일이 없어진다😨 그래서 검색 기능을 이용해 확장자를 검색해 보니 파일이 나타났다. 따라서 이 문제에 대해 이슈를 작성했다. 답변이 달리거나, 문제가 해결되면 이 글을 수정할 예정이다..!😃

💚이전글 읽기💚 이슈 등록 (Issue #7343):  https://cnu-devapp.blogspot.com/2020/11/nextcloudandroid-pin-issue-7343.html 버그로 오해받은 이슈가 다시 enhancement label로 수정되었다. #7343   그리고 멤버 개발자가 설명 고맙다고 말하면서 좋은 개선 사항이라 칭찬해줬다..ㅎㅎ😎 나... 칭찬 받았다 멤버 개발자의 의견을 듣고 나니 PR을 날려도 되겠다고 생각했다. tobiasKaminsky 멤버 개발자는 구현을 정말 빠르게하고 PR을 날린다. 그래서 지금 당장 코드 구현을 하지 않으면 코드 기여를 할 수 없을거란 조마조마한 마음에ㅋㅋㅋ 얼른 안드로이드 스튜디오를 열어 코드 분석을 하였다.  직전에 passcodelock.xml을 수정했기 때문에 이 레이아웃을 사용하는 클래스를 찾아봤다. PassCodeActuvuty.java에서 사용하고 있었다. 그래서 이 클래스를 살펴봤다. 코드를 보니까 passcode를 설정하거나 해제할때도 이 클래스를 사용했다. 처음에는 onBackPressed()를 수정하여 앱을 실행시켜봤다. 근데.. 뒤로가기가 안된다.. 너무 쉽게 생각했나..😂 그래서 클래스 내부함수를 다시 확인했고, 그 중 onKeyDown()에서 키이벤트 관련 코드들이 있었다.  1 2 3 4 5 6 7 8 9 10 11 @Override      public   boolean  onKeyDown( int  keyCode, KeyEvent event){          if  (keyCode  = =  KeyEvent.KEYCODE_BACK  & &  event.getRepeatCount() = =   0 ){ ...

 이번에도 Nextcloud 애플리케이션 사용 중 오류를 발견했다..! 애플리케이션에 업로드된 사진을 선택하고, 이를 연락처 사진으로 설정하고자 할 때 연락처 애플리케이션이 종료되는 것을 발견했다. 혹시나 앱 권한 때문에 그럴까 싶어 Nextcloud 애플리케이션에 주소록 권한을 주었지만, 고쳐지지 않았다..🙈 따라서 이를 토대로 이슈를 작성했다. 후에 멤버 개발자님이 로그캣을 알려달라고 댓글을 달아주셨다. github에 올라온 방법대로 로그를 보려고 했으나, 에러가 나서 리눅스 환경에서 로그를 출력하고 댓글에 첨부했다. 아직 문제가 해결되지 않았다. 나중에 문제가 해결된다면 다시 추가할 예정이다..!

  Nextcloud 애플리케이션 사용 중 에러가 발생했다😢 파일을 업로드하고 새로고침을 실행하자, 에러가 발생하며 Nextcloud 애플리케이션이 멈춰버렸다. 애플리케이션에서 "로그 + 관리자에게 보고하시겠습니까?" 라는 알림창이 떠서 Issue를 작성하게 되었다.  그리고 멤버 개발자님에게 답변을 받을 수 있었다. → 이것이 신뢰할 수 있는지에 대한 답변이었다. 예상치 못한 에러였기때문에 이에 대해 아직 답변하지 못했다. 이슈에서 찾아봤지만 나와 같은 에러를 발견하지 못했다. 😭 조금 더 살펴보고 답변을 해야할 것 같다..! +) 혹시라도 비슷한 에러나 좋은 답변이 있다면 말씀 해주시면 감사 드리겠습니다 ㅠㅠ

기존의 코드 기여에서  이슈관리를 통해 프로젝트에 기여하기로 방향을 바꾸고, 이슈를 찾아보던 중 … 같은 팀의 수빈님이 올리신 이슈를 발견했다.   자동업로드의 옵션을 변경할 수 있는지 삼성 기기를 사용하는  다른 사용자의 의견을 듣고 싶다고 댓글을 등록하셨다. 삼성 기기로 테스트 해보고 같은 오류가 발생한다는 것을 코멘트로 달았다.  처음으로 이슈에 코멘트를 달아봤다. 처음이라 무섭고 어려웠지만 막상 시도해보니, 프로젝트에 도움이 될 수 있다는 것이 신기했다! :)

 Migrations.kt 파일의 오타를 발견했다..! 파일에서 찾은 오타의 내용은 다음과 같다. 1.  Descriptionsss →Description 2.  addming → adding 두 가지의 오타를 찾고 이를 수정하여 PR을 날렸고, 머지를 받아 Contributor가 될 수 있었다..! 너무 뿌듯했다 😃

passcode관련 이슈 등록으로 앱에서 계속해서 확인하기 위해 새 코드를 빌드할 때 마다 passcode를 설정했다 해제했다는 자주 반복하며 테스트 하고 있다. 그런데 갑자기 passcode 화면이 안예쁘다!😕 뭐지... 무슨 느낌이지 했는데 알고 보니 레이아웃 상단 타이틀과 취소 버튼이 왼쪽 정렬이 되어버렸다.  원래는 중앙 정렬이었는 데,  #7338  이때 코드를 수정하면서 중앙 정렬이 적용되지 않은 것 같다. passcodelock.xml에서 중앙 정렬이 되도록 수정하였다. 그리고 현재는 PR을 날리고 기다리고 있는 상태이다! #7370 +) 테스트마다 결과가 따로 나와서 fail 메일이 많이 온다..헤헤... 수빈님 게시글을 보고 DCO 테스트 통과를 위해서는 Commit에 서명을 하면 된다는 사실을 알았다..❤  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

앱의 기능들을 이것저것 실행해보다가 발견한 오류이다! text document를 생성하고 글을 작성하고 있는 데 ' ‧‧‧'버튼이 아래로 내려와서 겹쳐졌다.😲 작성 화면 왼쪽상단에 겹친 '' ‧‧‧' 버튼 놀라서 작성을 멈추고 있었는 데 약 8초 후에 버튼이 다시 원래 자리로 돌아갔다.. 원래대로 돌아온 '' ‧‧‧' 버튼 그런데 애뮬레이터에서는 해당 오류가 발견되지 않았다. 그래서 은경님한테 부탁해서 테스트를 요청했더니 나와 같은 오류가 발생한다고 했다. 두명이나 발생한 오류면 이건 오류다! 한명만 발생하는 오류는 뭔가,,, 내 폰 문제일 것 같고 내 잘못일 것 같고,,, 그렇다😢 팀회의를 마치고 이슈로 등록하였다. #7354  💚 Nextcloud/android 멤버 개발자가 누구를 태그걸어 ping을 보냈다. 전수현 강사님께서 ping오면 pong하라고 하셨는데,,,ㅎㅎ 날 태그한게 아니니까,,, 알고보니까 Nextcloud/text 멤버 개발자를 부른 거였다. 이슈 올리기 전에 해당 코드를 찾아보려고 했는 데, 찾기가 너무 어려웠다. 그런데 이게 앱 이슈가 아니라 text 이슈였던 것이다.😅😅 멤버 개발자가 이 이슈를  Nextcloud/text로 옮겼다. #1186   나는 이제 Nextcloud 세개의 레포에 이슈를 등록하게 되었다ㅏ...하하 그리고 멤버 개발자가 Nextcloud 어떤 버전을 실행시켰는지 물어봤는데 내가 서버를 따로 만든들어서 사용하고 있는 게 아니라 어떤 버전을 사용하고 있는지 모른다..  사실대로 모른다고 이야기했다! 모르는 걸 어떡하냐! 찾아봐도 서버 버전 정보가 어디에 있는 지 모르겠다,, 그래서 버전을 알고있는 NC20 데모를 실행하여 문서를 작성했고 동일한 오류가 발생하는 것을 확인했다. 브라우저에서도 오류가 발생하는지 물어봐서 원래 사용하던 NC와 데모 모두 테스트 해보았다. 웹에서는 버그가 발생하지 않았다. 위 내용을 이슈 내용에 추가하였다. 생각지도 ...

 이 버그는 #7323과 #7305의 재생 바 오류를 해결한 개발자에게 테스트 요청이 들어와 테스트 하던 중에 발견한 오류다. 이슈 재현 단계 미디어 파일을 재생하다가 앱 최소화한 후 다시 들어가봤는데, 재생 바와 FAB가 겹치는 것이다. 처음엔 이게 오류가 아닌 줄 알았다..😮 그래서 내 환경에서는 FAB와 재생 바가 겹친다고 이슈 재현 영상과 함께 답변했다! 유나님이 찾아준 관련 이슈 알고보니, 이 오류는 PR #7339와 관련이 없으며 Issue #6782에 논의되었던 오류였다...! Andy Scherzings로부터 FAB는 이 화면에 나타나선 안되며, 어느 버전을 사용하고 있냐고 물었다. 또한 Issue #6782는 3.14.0 RC *에 이미 해결되었다고 알려줬다. 처음엔 Nextcloud QA version 7339에서 테스트를 해봤고, 3.14.0 RC 2에서도 테스트를 해봤다! 둘 다 동일하게 FAB 버튼이 나타난다. 사실 QA 버전에서 다른 기능을 테스트해도 되는지는 잘 모르겠다..ㅎㅎ 3.14.0 RC 2에서도 나타나는 오류니깐 넘어가자 😅😅 어쨌든 관련이 없는 이슈니, 여기서 더 언급하면 안될 것 같았고 #6782에 코멘트를 달겠다고 했다. Issue #6782는 아직 열려있는 이슈지만, 이미 이 이슈를 해결한 PR이 Merge된 상태다. Issue #6782에 단 코멘트  해결된 듯 싶지만 내 환경에서는 여전히 나타나는 오류라고 설명했다. 이에 대해 Andy Scherzings는 새로 이슈를 열어달라고 요청했고, 나는 #7356에 새롭게 리포트했다! 개발자 분들의 이해를 돕기 위해 이슈가 재현되는 영상을 함께 첨부했고,  관련 이슈 #6782도 언급했다! 이미 해결되어야 하는 이슈지만, 나에게 계속 발생한다고 작성했다😟 제출한 이슈 #7356 Andy의 답변!  tobias Kaminsky의 답변도 기다리고 있다~!