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

Create zamzar-sample-for-wamp #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

ask2rg
Copy link

@ask2rg ask2rg commented Aug 13, 2021

Working demo of zamzar API working on localhost over wamp server (windows). code written in PHP

Working demo of zamzar API working on localhost over wamp server (windows). code written in PHP
@whyleyc
Copy link
Owner

whyleyc commented Aug 24, 2021

@ask2rg Thanks for submitting this - I took a look and can see it adds some extra echo statements to the base code. I can see how this would be useful if you are running the PHP script in a hosted environment (e.g. via a web server), but the original code is designed just to showcase running from the command line.

I'll update the README to make this clear, and am also happy to link out to your code if you wish to host it in a separate repository (I won't integrate it here as I just wish to keep the sample code for command line only). Thanks.

@ask2rg
Copy link
Author

ask2rg commented Aug 24, 2021

Hi @whyleyc i also happy if you link out my code, you are completely free to use or distribute this code. i will more happy if you would able to give credit to me along with this
i use many echo so that a new user will able to see output of each step that's make easier to understand the flow of zamzar api works.
this sample code show how to use zamzar api with php in local environment of windows (localhost wamp server)

i am new to programing and php so sample code may have some mistakes, correct me where i am wrong

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