Skip to content
This repository was archived by the owner on Dec 21, 2021. It is now read-only.

Conversation

@teogeb
Copy link
Contributor

@teogeb teogeb commented May 10, 2021

Added constants StorageNode.STREAMR_GERMANY and StorageNode.STREAMR_DOCKER_DEV. The values are instances of StorageNode class.

The parameter of addToStorageNode, removeFromStorageNode and getStreamPartsByStorageNode can now be a StorageNode instance or a string.

@linear
Copy link

linear bot commented May 10, 2021

NET-286 Expose the address of Streamr Germany storage node as a constant

Note that there can be many storage nodes in the future. Also expose the streamr-docker-dev storage node with some name.

@teogeb teogeb requested a review from timoxley May 10, 2021 10:34
Copy link
Contributor

@timoxley timoxley left a comment

Choose a reason for hiding this comment

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

Yep this is much better. Worth updating readme? Otherwise lgtm.

@teogeb teogeb merged commit 2b01037 into master May 10, 2021
@teogeb teogeb deleted the NET-286-storage-node-constants branch May 10, 2021 17:41
bearni95 pushed a commit that referenced this pull request May 17, 2021
Added constants StorageNode.STREAMR_GERMANY and StorageNode.STREAMR_DOCKER_DEV. The values are instances of StorageNode class.

The parameter of addToStorageNode, removeFromStorageNode and getStreamPartsByStorageNode can now be a StorageNode instance or a string.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants