devapp - Contributing Nextcloud Open Source

우리 devapp 팀원들은 모두 Nextcloud/android의 Contributor가 되었다!😎 아래는 각 팀원이 Nextcloud/android의 Contributor임을 증빙하는 자료이다. ✨ 김수빈 ✨ 권유나 ✨ 권은경 ✨ 김민지 ✨ 이인재

    💚이전글 읽기💚 이슈 등록 (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   Merge:  https://cnu-devapp.blogspot.com/2020/11/nextcloudandroid-passcode-pr-merge_25.html 검색이슈를 해결하고 싶어서 Nextcloud/server Repository를 살펴보다가 README 파일을 읽게 되었다. 여기에 기여에 관한 설명도 나와있었는데, 모든 기여자들은 기여한 파일 header comment에 copyright을 추가하라 는 말이 있었다. Nextcloud/android Repository에서도 이 글을 읽었던 것 같아서 확인해보니 동일한 설명이 android에도 있었다. 저번 코드 기여했을 때 copyright을 작성하지 않고 merge가 되었는데,,늦었지만 나도 copyright을 추가하면 좋겠다고 생각했다. 여기에 몇가지 고민사항이 있었다.  첫번째로 나는 하나의 클래스나 하나의 함수를 새로 구현한 것이 아닌 기존에 있던 함수의 일부로 새 기능을 추가 구현한 것 인데, 이것도 copyright 추가를 할 수 있을까?🤔 두전째는 몇가지 파일을 열어서 확인해보니 member거나 collaborator만 copyright을 적는 것 같았다.😥 세번째로 나는 이미 merge가 되었는 데, copyright comment만 추가하는 걸 멤버개발자들이 merge해줄지 걱정되었다.😩 12월 8일 6시에 공개소프트웨어실습 이홍로 교수님과 면담 형식의 개인 발표를 진행했는데, 이때 이러한 고민을 이야기하고 조언을 받았다. 교수님께서는 먼저 멤버개발자에게 물어보는것을 권하셨다. 그래서 개인...

   💚이전글 읽기💚 이슈 등록 (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())  | |   ...

  💚이전글 읽기💚 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는 디자인을 추가로 수정하는 ...

💚이전글 읽기💚 이슈 등록 (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 ){ ...

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와 데모 모두 테스트 해보았다. 웹에서는 버그가 발생하지 않았다. 위 내용을 이슈 내용에 추가하였다. 생각지도 ...

민지님 이슈를 확인하면서 앱에 대해 불편함 점을 발견했다. 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

검색 이슈를 등록한 후 내가 코드로 구현할 수 있는 다른 이슈들이 있는 지 확인하던 중 #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

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

💚이전글 읽기💚 검색이슈 pt.1:  https://cnu-devapp.blogspot.com/2020/11/nextcloudandroid-pt1.html 서버가 NC19에서 NC20으로 버전 업데이트를 하면서 통합된 검색 기능이 생겨났다. 그리고 멤버 개발자가 안드로이드 앱도 3.15버전으로 업데이트하면서 통합된 검색기능을 사용할 것이고 이로 인하여 검색이슈가 해결될 것이라 말했다. 그리고 NC20 웹에서 이 이슈를 확인할 수 있는 지를 물어봐서 이슈를 확인해보았다. NC20버전을 사용하기위해 데모버전으로 실행하였다. [검색 성공] 파일명 순서 흐름대로 검색 [검색 실패] 파일명 순서 흐름대로가 아닌 중간 단어 생략 후 검색 [검색 성공]파일명 순서 흐름대로가 아닌 중간 단어 생략 후 검색 + 공백대신 '%'기호 사용 검색 이슈가 아직 해결되지 않았다. 멤버 개발자가 말한 통합된 검색은 파일이나 연락처 등을 동시에 검색할 수 있는 기능인것 같다. 그런데 웹에서도 이런 문제가 발생하는 거면 서버에서 해결해야 할 문제인 것 같다.😭 서버 구축을 아직 안했는데,, 시작해야겠다! 웹에서 테스트한 것은 이슈 내용으로 추가했다.  #7325 una.kwon Github : https://github.com/Kuuuna98 Nextcloud/android Github : https://github.com/nextcloud/android