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

node.js client should accept serverRendering and clientRendering endpoints instead of generic array (breaking change) #178

Merged
merged 9 commits into from
Jan 29, 2016

Conversation

matteofigus
Copy link
Member

There are 3 main breaking changes here.

  • new Client() is not accepted anymore. Options need to be passed in during initialisation: new Client({ //options });
  • follows that client.config is deprecated and made private and not accessible anymore
  • When initialising with the options, { registries: []} as array is deprecated in favour of an object that contains 0, 1 or both keys of clientRendering and serverRendering. So, when doing server-side rendering with failover, failover request will be able to have client-side rendering friendly url.

@matteofigus matteofigus changed the title [don't merge yet] Various node.js client improvements node.js client should accept serverRendering and clientRendering endpoints instead of generic array (breaking change) Jan 28, 2016
@matteofigus matteofigus changed the title node.js client should accept serverRendering and clientRendering endpoints instead of generic array (breaking change) [don't merge yet] node.js client should accept serverRendering and clientRendering endpoints instead of generic array (breaking change) Jan 28, 2016
@matteofigus matteofigus changed the title [don't merge yet] node.js client should accept serverRendering and clientRendering endpoints instead of generic array (breaking change) node.js client should accept serverRendering and clientRendering endpoints instead of generic array (breaking change) Jan 28, 2016
@andyroyle
Copy link
Collaborator

lol, travis is trying to lint the minified files

jankowiakmaria added a commit that referenced this pull request Jan 29, 2016
node.js client should accept serverRendering and clientRendering endpoints instead of generic array (breaking change)
@jankowiakmaria jankowiakmaria merged commit cf240f0 into master Jan 29, 2016
@jankowiakmaria jankowiakmaria deleted the node-client-improvements branch January 29, 2016 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants