Skip to content

Conversation

@ggmaleva
Copy link
Contributor

No description provided.

Comment on lines 15 to 16
identity:
$ref: "../schemas/IdentityID.yaml"
Copy link

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.

это поле мы сохраняем чтобы позже дропнуть или как? мы же не сможем его поддерживать теперь

Я его сохранил для текущих клиентов, которые работают со старой версией fistful.
То есть да - сохраняем, чтобы потом дропнуть.
Более плавный переезд)

Copy link

Choose a reason for hiding this comment

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

это поле мы сохраняем чтобы позже дропнуть или как? мы же не сможем его поддерживать теперь

Я его сохранил для текущих клиентов, которые работают со старой версией fistful. То есть да - сохраняем, чтобы потом дропнуть. Более плавный переезд)

Вопрос только что дает плавный переезд, в одной версии будет 1 поле которое есть, но не функционирует, а для другой части будет другое поле, которе есть, но не функционирует. Если бы мы интегрировались что бы мы почувствовали?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ну и я deprecated добавил с поясняющим комментом, чтобы клиенты могли плавно перейти

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

Choose a reason for hiding this comment

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

Ну и я deprecated добавил с поясняющим комментом, чтобы клиенты могли плавно перейти

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

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