Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show no submissions message when there is no attempts left #339

Merged
merged 2 commits into from
Aug 10, 2018

Conversation

ivan-magda
Copy link
Member

Задача: #APPS-1888

Описание:
Отображение сообщения о том что попытки закончились в шаге с ограниченными попытками.

simulator screen shot - iphone se - 2018-08-06 at 16 22 33

@ivan-magda ivan-magda added the main label Aug 6, 2018
@ivan-magda ivan-magda added this to the 1.65 milestone Aug 6, 2018
@ivan-magda ivan-magda self-assigned this Aug 6, 2018
@ivan-magda ivan-magda requested a review from Ostrenkiy August 6, 2018 14:06
@kvld kvld self-requested a review August 10, 2018 11:30
@@ -333,21 +335,25 @@ class QuizPresenter {
[weak self]
submission in

guard let s = self else { return }
guard let `self` = self else {
Copy link
Contributor

Choose a reason for hiding this comment

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

Предлагаю использовать strongSelf для такого обозначения. Так можно сразу понимать, о чем речь.

Copy link
Member Author

Choose a reason for hiding this comment

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

Хорошо. Я по-моему изначально подумал, что s это submission или любой другой объекто, но точно не self :trollface:

@ivan-magda ivan-magda merged commit 6874017 into dev Aug 10, 2018
@ivan-magda ivan-magda deleted the fix/no-submissions-left branch August 10, 2018 16:20
@kvld kvld mentioned this pull request Aug 23, 2018
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants