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

v1.3.10 - Added getProfile and updateProfile commands to provision proxy #65

Merged
merged 3 commits into from
Aug 9, 2018

Conversation

tsmith1985
Copy link
Contributor

No description provided.

@tsmith1985 tsmith1985 requested review from rmomii and ggentzel August 7, 2018 23:52
Copy link
Contributor

@rmomii rmomii left a comment

Choose a reason for hiding this comment

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

other than the history.md and package.json merge conflicts, it looks good

Copy link
Contributor

@ggentzel ggentzel left a comment

Choose a reason for hiding this comment

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

Just looks like it needs to pull from develop to get sync'd and a question about why the package version skipped over a patch increment

package.json Outdated
@@ -1,6 +1,6 @@
{
"name": "playnetwork-sdk",
"version": "1.3.8",
"version": "1.3.10",
Copy link
Contributor

Choose a reason for hiding this comment

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

why did we skip over 1.3.9 here?

Copy link
Contributor

Choose a reason for hiding this comment

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

disregard this comment and the thing about needing pulling. i see what happened there now. i think just the one comment below that order should probably be profile copypaste issue need addressing

lib/provision.js Outdated
if (typeof options === 'function') {
callback = options;
options = undefined;
if (typeof order === 'function') {
Copy link
Contributor

Choose a reason for hiding this comment

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

should order be profile instead, copypasta?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

😜

@tsmith1985 tsmith1985 merged commit 8bd09a1 into develop Aug 9, 2018
@tsmith1985 tsmith1985 deleted the v1.3.10 branch August 9, 2018 16:20
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.

3 participants