Skip to content
This repository has been archived by the owner on Oct 14, 2021. It is now read-only.

Show progress feedback to user on download something #84

Open
seompark opened this issue Mar 4, 2019 · 12 comments
Open

Show progress feedback to user on download something #84

seompark opened this issue Mar 4, 2019 · 12 comments

Comments

@seompark
Copy link
Member

seompark commented Mar 4, 2019

AJAX로 파일 다운로드를 하게되면 사용자가 요청한 뒤에 아무런 반응이 없고 그저 하염없이 기다려야만 하기 때문에 적절한 UI 피드백이 필요합니다 (해당 버튼에 다운로드 퍼센트를 보여주는 등)

해당하는 서비스 목록

  • 인강실 관리자 액셀 다운로드
  • 과제 관리자 과제 다운로드

추가바람

@taevel02
Copy link
Member

확인해본 결과 도서 관리자 엑셀 다운로드와 Dets 관리자 엑셀 다운로드, 상담 관리자 엑셀 다운로드도 해당되는 것 같아 이 서비스 또한 ui 피드백을 진행하겠습니다.

@uhmseohun
Copy link
Member

페이지 로딩 할 때 처럼 NProgress로 할까요?

@taevel02
Copy link
Member

전 과제 다운로드 처럼 텍스트로 퍼센트를 보여주는게 이쁠 것 같아 그렇게 했었는데 NProgress도 괜찮을 것 같아요.

@uhmseohun
Copy link
Member

uhmseohun commented Mar 23, 2019


일단 자고 내일 할게요ㅠㅠ

@uhmseohun
Copy link
Member

#169 덕분에 다 NProgress 적용됬는데 이슈 닫을까요?

@taevel02
Copy link
Member

taevel02 commented Mar 24, 2019

NProgress가 적용되어 있지 않습니다. 해당 이슈는 퍼센트를 보여주는 등으로 다운로드 현황을 보여주는 것이 목적이에요.

#169 는 권한 여부를 체크해서 엑셀 다운로드를 가능하게 하는 것이 목적인 pr입니다

@uhmseohun uhmseohun reopened this Mar 24, 2019
@uhmseohun
Copy link
Member

@uhmseohun
Copy link
Member

퍼센트로 구현해볼게요

@uhmseohun
Copy link
Member

uhmseohun commented Mar 25, 2019

스크린샷 2019-03-25 12 48 41

lengthComputablefalseloaded / total을 계산할 수가 없어서 퍼센트 구현이 어려울 것 같아요

@uhmseohun
Copy link
Member

이슈 닫을게요

@seompark
Copy link
Member Author

lengthComputable이 false라는건 서버에서 Content-Length 헤더를 안 보내준다는건데 그럼 그걸 해결해야죠

@uhmseohun
Copy link
Member

서버에서 구현되면 바로 진행 할게요

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants