Skip to content

Conversation

@Robalim
Copy link
Contributor

@Robalim Robalim commented Nov 21, 2023

import org.threeten.bp.Instant

/**
* An interface for internal sdk work only. Do not implement or use it
Copy link
Collaborator

Choose a reason for hiding this comment

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

An interface for internal sdk work only. Do not implement or use it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Что-то не так с комментом?

Copy link
Collaborator

Choose a reason for hiding this comment

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

точка в конце

/**
* A class for internal sdk work only. Do not implement it
* */
data class PushAction(
Copy link
Collaborator

Choose a reason for hiding this comment

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

а почему это публичное?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

потому что цепочка зависимостей ведет к интерфейсу MindboxImageFailureHandler, который должен быть публичным

Copy link
Collaborator

Choose a reason for hiding this comment

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

Тогда согласен

Copy link
Collaborator

Choose a reason for hiding this comment

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

А нет Do not implement it. Data класс все равно нельзя наследовать. Путь имплементируют, если захотят свою реализацию пушей

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Если они захотят свою реализацию пушей, пусть пишут свою реализацию пушей? Зачем им наши модельки? Просто чтобы свои не создавать по необходимости?

Copy link
Collaborator

Choose a reason for hiding this comment

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

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

поправил комменты

@Robalim Robalim merged commit ef7ba4e into develop Nov 23, 2023
@Robalim Robalim deleted the feature/MBX-2934 branch November 23, 2023 12:04
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.

3 participants