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

Feat(samples)print request response #13

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

akshunyaInfy
Copy link
Contributor

@akshunyaInfy akshunyaInfy commented Jan 30, 2019

  1. Printing Request and Response details in each sample code.
  2. Removal of unnecessary log statements in SampleCode.cs file.
  3. Sample Codes display logic has been improved.
    Before this fix, whenever any sample code was run, after showing the response the full list of sample codes was displayed which used to hide the response and user had to scroll up to see the response.
    The fix includes asking for a Readline input from user after displaying the response to hold the screen. After pressing any key, again the full list is displayed.
  4. Few comments have been added to help understand the flow of the logic mentioned in the point#3.

Note: This PR has a dependency on the PR of Client SDK and should be merged only after the client SDK is released in the public repo with all the latest changes.

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