Skip to content

More details about SimSwap in the API documentation#82

Merged
fernandopradocabrillo merged 4 commits intocamaraproject:mainfrom
gregory1g:main
Dec 11, 2023
Merged

More details about SimSwap in the API documentation#82
fernandopradocabrillo merged 4 commits intocamaraproject:mainfrom
gregory1g:main

Conversation

@gregory1g
Copy link
Contributor

More explicit definition of SimSwap in the API description.

More detailed description of scenarios where SimSwap happens and API can be used.

What type of PR is this?

documentation

Which issue(s) this PR fixes:

- More explicit definition of SimSwap in the API description

- More detailed description of scenarios where SimSwap happens and  API can be used.
Copy link

@mhfoo mhfoo left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@fernandopradocabrillo fernandopradocabrillo left a comment

Choose a reason for hiding this comment

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

Thanks @gregory1g for the PR.

Just a couple suggestions, the rest is fine for me

bigludo7
bigludo7 previously approved these changes Dec 1, 2023
Copy link
Collaborator

@bigludo7 bigludo7 left a comment

Choose a reason for hiding this comment

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

LGTM

ok

Co-authored-by: Fernando Prado Cabrillo <pradocabrillo.fernando@gmail.com>
gregory1g and others added 2 commits December 1, 2023 14:42
Co-authored-by: Fernando Prado Cabrillo <pradocabrillo.fernando@gmail.com>
@gregory1g
Copy link
Contributor Author

Updated following @fernandopradocabrillo suggestions.

Copy link
Collaborator

@fernandopradocabrillo fernandopradocabrillo left a comment

Choose a reason for hiding this comment

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

LGTM

@fernandopradocabrillo fernandopradocabrillo merged commit 8496063 into camaraproject:main Dec 11, 2023
@fernandopradocabrillo
Copy link
Collaborator

@gregory1g merged since it has 2 codeowner approvals

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.

4 participants