Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update(27062): SNA URL request updated to POST #7

Merged
merged 4 commits into from Sep 22, 2022

Conversation

mejiagarcia
Copy link
Contributor

@mejiagarcia mejiagarcia commented Sep 15, 2022

Ticket

  • [27062]

Description

  • SNA URL request updated to POST request
  • Sample app updated to show SDK version

Commit message

  • update(27062): SNA URL request updated to POST

Screenshot

Screen Shot 2022-09-15 at 9 44 49 AM

Testing

  • Added unit tests
  • Ran unit tests successfully
  • Added documentation for public APIs and/or Wiki

Copy link
Contributor

@AlejandroOrozco AlejandroOrozco left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@mejiagarcia mejiagarcia merged commit 60bbb56 into main Sep 22, 2022
@mejiagarcia mejiagarcia deleted the cmejia/27062/post branch September 22, 2022 19:18
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.

None yet

3 participants