Update quickstart guide for new PHP SDK#4
Uh oh!
There was an error while loading. Please reload this page.
Conversation
ssung88force-pushed the ssung/update_quickstart branch 3 times, most recently from a7ee794 to 403e2feComparessung88 changed the title 
quickstart/composer.json Outdated
| "require":{ | ||
| "vlucas/phpdotenv": "^5.6" | ||
| "vlucas/phpdotenv": "^5.6", | ||
| "square/square": "^41.0.0" |
There was a problem hiding this comment.
My only point of feedback is that for these quickstart samples I don't want to lock the version into the composer.json. I just want the README to tell the user to install the latest version of the SDK, so we don't have to keep updating this sample manually when new versions release every month.
So given that, we can merge this PR when the new SDK goes live, and the 'install latest' method will still work.
Open to dissenting thoughts on this though
There was a problem hiding this comment.
Good catch! What do you think about requiring >=41.0.0 instead? The new SDK changes start in 41, ensuring compatibility.
There was a problem hiding this comment.
That sounds good, lets do that!
ssung88force-pushed the ssung/update_quickstart branch from 403e2fe to 2b28b7dCompare
3 participants
Uh oh!
There was an error while loading. Please reload this page.