Skip to content

Статусы сообщений#35

Open
Kozlov-AE wants to merge 3 commits intomasterfrom
feature/#14-message-status
Open

Статусы сообщений#35
Kozlov-AE wants to merge 3 commits intomasterfrom
feature/#14-message-status

Conversation

@Kozlov-AE
Copy link
Copy Markdown
Contributor

No description provided.

@Kozlov-AE Kozlov-AE linked an issue May 7, 2021 that may be closed by this pull request
4 tasks
tranee95
tranee95 previously approved these changes May 13, 2021
Copy link
Copy Markdown
Collaborator

@tranee95 tranee95 left a comment

Choose a reason for hiding this comment

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

код проверил, работает, есть небольшие пожелания, но это не кретично

SavkinIS
SavkinIS previously approved these changes May 16, 2021
@Kibnet Kibnet added the Улучшение Новая фича label May 16, 2021
Copy link
Copy Markdown
Contributor

@Kibnet Kibnet left a comment

Choose a reason for hiding this comment

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

Да, оно работает! Но код требует серьёзной доработки. Допущен ряд ошибок которые в дальнейшем могут привести к плавающим багам. Детали в комментариях.

@Kozlov-AE Kozlov-AE dismissed stale reviews from SavkinIS and tranee95 via dcfb3cd June 16, 2021 06:43
@Kibnet Kibnet force-pushed the feature/#14-message-status branch 2 times, most recently from 75980b5 to af525bc Compare June 16, 2021 10:18
@Kozlov-AE Kozlov-AE force-pushed the feature/#14-message-status branch from 365640b to 4b2bfcd Compare June 16, 2021 13:28
@Kozlov-AE Kozlov-AE force-pushed the feature/#14-message-status branch 2 times, most recently from ffe8561 to 714e6bf Compare July 24, 2021 21:06
@Mileeena Mileeena self-requested a review July 25, 2021 07:38
@Kozlov-AE Kozlov-AE requested review from EvgeniyLyapunov and removed request for ARuslanRu, Mileeena and vvn62 July 26, 2021 15:32
@Kozlov-AE Kozlov-AE force-pushed the feature/#14-message-status branch 2 times, most recently from 32cbb99 to 800edcb Compare July 26, 2021 21:17
Вынес присвоение статусов моим сообщениям из цикла (похоже опечатался, когда туда его засунул)
Переименовал метод
Поправил прыгающие галочки
Переименованы статусы "прочитано"
Поправлена логика обнаружения нижнего видимого сообщения
Поправлен конвертер для расстановки галочек
@Kozlov-AE Kozlov-AE force-pushed the feature/#14-message-status branch from 800edcb to 1f7ff64 Compare July 31, 2021 19:31
@Kibnet Kibnet changed the title Feature/#14 message status Статусы сообщений Aug 4, 2021
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.

Статус сообщения

5 participants