Skip to content

Conversation

@6RUN0
Copy link
Contributor

@6RUN0 6RUN0 commented Nov 1, 2025

Добавил методы для добавления, удаления контакта из списка контактов

Тип изменений

  • Исправление бага
  • Новая функциональность
  • Улучшение документации
  • Рефакторинг

@ink-developer
Copy link
Owner

В общем хорошо, спасибо, но

  1. просьба все обернуть в try except и добавить логи (и обработку ошибок из пейлоада соответственно)
  2. Ошибка с тайпингом, remove contact всегда возвращает True, а это не верно, ведь может быть и ошибка. И ещё add contact может вернуть None, нужно тоже указать в тайпинге

@6RUN0
Copy link
Contributor Author

6RUN0 commented Nov 1, 2025

  1. Может решение о логгировании оставить разработчику стороннего приложения, на основе этой библиотеки. А в самой библиотеке сделать четкую структуру исключений и вообще убрать обёртку try except, где это не нужно?

  2. Если ошибка - будет выброшено исключение. Можно вообще поменять на None, там payload в ответе пустой

Добавление контакта

[OUTGOING] {"ver":11,"cmd":0,"seq":43,"opcode":34,"payload":{"contactId":11111111,"action":"ADD"}}
[INCOMING] {"ver":11,"cmd":1,"seq":43,"opcode":34,"payload":{"contact":{...}}}

Удаление контакта

[OUTGOING] {"ver":11,"cmd":0,"seq":38,"opcode":34,"payload":{"contactId":11111,"action":"REMOVE"}}
[INCOMING] {"ver":11,"cmd":1,"seq":38,"opcode":34,"payload":{}}

@ink-developer
Copy link
Owner

после обновы чуть по подробнее разберу

@ink-developer
Copy link
Owner

Извиняюсь за столь долгое отсутствие

@ink-developer
Copy link
Owner

просьба решить конфликты веток и я принимаю

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants