Skip to content
This repository was archived by the owner on Jan 7, 2020. It is now read-only.

Conversation

@bochaco
Copy link
Contributor

@bochaco bochaco commented Mar 11, 2019

Adding getXorUrl experimental function to the ImmutableData Reader which generates the XOR-URL of an already existing and fetched ImmutableData. Up until now you can get ImmD XOR-URL only when storing it on the network, with this you can get it after fetching it with its xorname.

Resolves #353

@coveralls
Copy link

coveralls commented Mar 11, 2019

Coverage Status

Coverage increased (+0.01%) to 96.499% when pulling 9107bbb on bochaco:issue-353-immd-xor-url into 32d7a6e on maidsafe:master.

hunterlester
hunterlester previously approved these changes Mar 12, 2019
* try {
* const cipherOpt = await app.cipherOpt.newPlainText();
* const iDataWriter = await app.immutableData.create()
* const data = 'Most proteins are glycosylated.
Copy link
Contributor

Choose a reason for hiding this comment

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

For multiline text, I think ``` is cleaner that ' (github is flagging this up as being an error, eg).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

let me take a look a this, and correct all docs that use' if it works with your suggestion

joshuef
joshuef previously approved these changes Mar 13, 2019
bochaco added 2 commits March 13, 2019 12:51
…eData Reader which generates the XOR-URL of an already existing and fetched ImmutableData
@bochaco bochaco dismissed stale reviews from joshuef and hunterlester via 9107bbb March 13, 2019 15:52
@bochaco bochaco force-pushed the issue-353-immd-xor-url branch from d9f99ef to 9107bbb Compare March 13, 2019 15:52
@bochaco bochaco merged commit d4cf6e5 into maidsafe-archive:master Mar 13, 2019
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.

4 participants