Skip to content

Conversation

Dovchik
Copy link
Contributor

@Dovchik Dovchik commented May 13, 2025

Closes https://tickets.sinch.com/browse/DEVEXP-901

README.md snippets wll be updated in a separate PR

@Dovchik Dovchik changed the base branch from main to v2.0-next May 13, 2025 12:49
@Dovchik Dovchik requested a review from samwil-sinch May 13, 2025 12:49
@Dovchik Dovchik marked this pull request as ready for review May 13, 2025 12:54

Choose a reason for hiding this comment

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

Can you also add a paragraph for the deprecated endpoints?

Choose a reason for hiding this comment

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

Can you also add some text explaining the rationale behind this refactoring?


If you are coming from version 1.*, you can see how to use

Choose a reason for hiding this comment

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

Is this paragraph complete? It feels like this sentence is missing some parts

# Migration to 2.0

## Content list:
- [Initialize with unified credentials](#initialize-sinchclient-with-unified-credentials)

Choose a reason for hiding this comment

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

The TOC title doesn't match with the paragraph title

@Dovchik Dovchik marked this pull request as draft May 14, 2025 11:35
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