Skip to content

Conversation

scottinet
Copy link
Contributor

@scottinet scottinet commented Sep 20, 2016

Instead of being read-only, the host, ioPort and wsPort attributes from the Kuzzle object should be writable to allow changing them between connections.
And when invoking Kuzzle.connect, if a network connection has previously been established, then Kuzzle.disconnect is called beforehand.

@scottinet scottinet added the wip label Sep 20, 2016
@codecov-io
Copy link

codecov-io commented Sep 20, 2016

Current coverage is 100% (diff: 100%)

Merging #115 into master will not change coverage

@@           master   #115   diff @@
====================================
  Files          16     16          
  Lines        1550   1552     +2   
  Methods       264    264          
  Messages        0      0          
  Branches      398    398          
====================================
+ Hits         1550   1552     +2   
  Misses          0      0          
  Partials        0      0          

Powered by Codecov. Last update f861454...7abd7ae

@scottinet scottinet removed the wip label Sep 21, 2016
@scottinet scottinet changed the title KUZ-683 allow changing host [MASTER] KUZ-683 allow changing host Sep 21, 2016
@scottinet
Copy link
Contributor Author

This PR merges the changes into the master branch, to allow the BO team to implement multi-environments support.

@dbengsch
Copy link
Contributor

Shouldn't ioPort and wsPort be writable as well ?

@scottinet
Copy link
Contributor Author

scottinet commented Sep 21, 2016

@dbengsch > saw that too while updating the Android SDK :p

@stafyniaksacha stafyniaksacha merged commit fe0af5b into master Sep 21, 2016
@scottinet scottinet deleted the kuz-683-allow-changing-host branch September 21, 2016 11: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.

6 participants