Skip to content

feat(attach): add progress bar #865

Merged
merged 4 commits into from
Nov 5, 2019

Conversation

sklyanchuk
Copy link
Contributor

Отслеживание процесса загрузки файла через индикатор выполнения (progress bar).

Мотивация и контекст

Я как пользователь, загружая тяжелый файл, хочу понимать, в каком состоянии сейчас загрузка моего файла и сколько мне, предположительно, ждать окончания загрузки.

@coveralls
Copy link

coveralls commented Oct 23, 2019

Coverage Status

Coverage increased (+0.01%) to 76.448% when pulling 28841b7 on feature/attach-with-progress-bar into 07c571b on master.

Copy link
Contributor

@stepancar stepancar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sklyanchuk @Heymdall @SiebenSieben а вот такие штуки не хотим заворачивать в один объект?
Или в данном случае вобще можно понять undefind или нет в progressBarPercent и понять показывать или нет

@sklyanchuk
Copy link
Contributor Author

sklyanchuk commented Oct 24, 2019

Или в данном случае вобще можно понять undefind или нет в progressBarPercent и понять показывать или нет

вообще согласен, если можно обойтись одним пропсом, то лучше так и сделать.
как сейчас принято в репе? есть какое-то единообразие решений?

@SiebenSieben
Copy link
Contributor

SiebenSieben commented Oct 25, 2019

Согласен, в данном случае лучше обойтись одной пропсой

Copy link
Contributor

@stepancar stepancar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Мне ок. jemini тестов больше не пишем?

@SiebenSieben
Copy link
Contributor

Да, gemini хорошо бы, добавил в бэклог

@SiebenSieben SiebenSieben merged commit fe15c5d into master Nov 5, 2019
@sklyanchuk sklyanchuk deleted the feature/attach-with-progress-bar branch November 5, 2019 15:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants