search

sokcuri / TweetDeckPlayer

Web-App Application for Useful and Better Tweet Life
Other
66 stars 21 forks source link

최신 업데이트 알림 기능 추가 #118

Closed ikasty closed 6 years ago

ikasty commented 6 years ago

업데이트가 있는지 매번 체크하기 귀찮아서, 업데이트 알림 기능을 추가했습니다. 자동 업데이트까지는 구현하지 않았습니다.

깃허브 releases 주소를 링크해주는 것도 생각해봤는데 마땅한 UI가 생각나지 않아서 일단은 팝업으로 구현해 두었습니다. 이 부분은 좀더 고민해 봐야겠네요.

ikasty commented 6 years ago

4 에 관련 이슈가 있습니다.

sokcuri commented 6 years ago

안녕하세요! 소스 기여를 해주셔서 감사합니다. 깃허브 API를 이용해서 업데이트 확인 여부를 구현한건 정말 생각치도 못했던 방법인데 흥미롭네요.

ikasty commented 6 years ago

디폴트로 업데이트 체크를 하도록 만드는 것도 물론 생각해 봤는데요, 그 부분은 좀 테스트를 더 해보고 싶었어요. 혹여나 업데이트 체크 기능때문에 특정 머신에서는 처음 어플리케이션 로딩 시간이 너무 오래 걸리지 않을까 해서 말이죠. 제 컴퓨터에서는 아무 문제가 없었지만요...

리뷰해주신 부분은 오늘 밤이나 내일 정도에 좀 고쳐볼게요 :D

ikasty commented 6 years ago

그리고 Toast로 업데이트를 알려주는 것까지는 좋은데, 업데이트를 좀 더 간편하게 할 만한 아이디어가 있을까요? 최소한 업데이트가 필요할 때 업데이트 주소 정도는 사용자에게 링크로 알려주고 싶은데...

sokcuri commented 6 years ago

깃허브에 업데이트된 버전의 릴리즈 정보는 https://api.github.com/repos/sokcuri/tweetdeckplayer/releases/latest 에서 가져올 수 있네요! 이번에 Release API가 있다는것도 처음 알았습니다. 아마 이쪽 정보를 가져오면 문제없이 사용이 가능할것같네요

토스트 아닌 방법은.. 조금 고민이 되네요. 저도 반디집 업데이트하라고 하면 귀찮아해서 ㅋㅋ 토스트에 링크가 안걸릴것같지만 걸리게 만들 수 있다면야 괜찮을것같네요.

출처 : https://developer.github.com/v3/repos/releases/#get-a-single-release

sokcuri commented 6 years ago

속도 문제는 그리 걱정할필요가 없는게 버전 체크 함수에서 쓰이는 http.get이 비동기이기 때문에 괜찮아요 굿!

ikasty commented 6 years ago

뭐 저도 혹시나싶어서 setTimeout 으로 한 번 묶어두긴 했지만요ㅎㅎ그럼 디폴트로 사용하도록 해둘게요.

릴리즈 정보는 현재도 링크해주신 주소를 토대로 구현되어 있습니다. 가장 최신 버전의 tag 이름을 기반으로 single release만 가져오고 있는데요, 테스트해 보니 토스트에 링크는 걸리지 않더군요. 대신 node의 os 패키지를 사용하면 현재 구동되고 있는 운영체제 버전 등을 알아낼 수 있을테니, 자동 업데이트 기능도 추가할 수 있겠네요. 이 기능에 대해선 별도로 풀 리퀘스트를 작성해야 할 거 같습니다.

sokcuri commented 6 years ago

테스트 후 이상 없음 확인했습니다. 수고하셨어요!