Update Readme [Future]#81
Conversation
Updated README for consistency with the new model of the sdk.
There was a problem hiding this comment.
I don't think we need to refer to old model in the updated readme. We should keep the old usage examples but just put a readme in the top of the old model folders and clearly say this is deprecated funtionality.
|
Thank-you for the updates, they look great just a couple of small comments. |
Removed references to the old model SDK. Also removed Apple Pay sample, as advised by @ramittal ('reference the sample-code so that it is always up-do date and verified functional'). But, I thought one sample ('Quick Usage Example' here) should be kept to illustrate the setup.
Removed details other than the sample code and its dependencies.
Correct formatting.
|
Hi @brianmc I've done the changes you advised:
Please give this a look. Also please confirm if (/lib/) is the right place to keep the old samples. Thanks, |
Trimmed usage example section to only keep the links to the dev center sample codes.
|
Thanks Akash, I would still like to see a very simple example in the usage section (e.g. the very simplest charge credit card with only the required fields) but also reference the samples repo. Thanks, Brian |
Added a quick usage example (charge credit card).
Section for logging details added. Necessary information about logging like defining regexes in configuration, file paths, and logging and masking usage is mentioned.
2 participants
(Note that this is in the Future branch)
Please review the updated README.
READMEs in SDKs of other languages would be updated in the same format once this is approved.
Thanks,
Akash Amat (aamat, visa)