Raspberry tutorial review #11

Merged
merged 46 commits into from Aug 6, 2015

Conversation

Projects
None yet
3 participants
Contributor

desrever-nu commented Aug 3, 2015

I reviewed the tutorial and I believe is not ready for a merge

This comment has been minimized.

Show comment Hide comment
@CoinGame

CoinGame Aug 3, 2015

Contributor

I unfortunately won't have time to run through and test the accuracy of the information. Have you run this PR by anyone on the forum that has done the setup, or did you do this all yourself while writing it?

Contributor

CoinGame commented Aug 3, 2015

I unfortunately won't have time to run through and test the accuracy of the information. Have you run this PR by anyone on the forum that has done the setup, or did you do this all yourself while writing it?

This comment has been minimized.

Show comment Hide comment
@brossi

brossi Aug 4, 2015

Contributor

I reviewed the tutorial and I believe is not ready for a merge

Not or Now?

Contributor

brossi commented Aug 4, 2015

I reviewed the tutorial and I believe is not ready for a merge

Not or Now?

This comment has been minimized.

Show comment Hide comment
@desrever-nu

desrever-nu Aug 4, 2015

Contributor

is "now" ready - sorry @brossi

Contributor

desrever-nu commented Aug 4, 2015

is "now" ready - sorry @brossi

This comment has been minimized.

Show comment Hide comment
@desrever-nu

desrever-nu Aug 5, 2015

Contributor

Have you run this PR by anyone on the forum that has done the setup, or did you do this all yourself while writing it?
@CoinGame , I have followed myself the steps while compiling .

Contributor

desrever-nu commented Aug 5, 2015

Have you run this PR by anyone on the forum that has done the setup, or did you do this all yourself while writing it?
@CoinGame , I have followed myself the steps while compiling .

This comment has been minimized.

Show comment Hide comment
@desrever-nu

desrever-nu Aug 5, 2015

Contributor

Ok, I ran through it, applied some changes and waiting for others to comment ...
Before merging, a couple of things still are needed ...

@CoinGame @brossi

  • from daemon how can I list my receive NSR addresses?
Contributor

desrever-nu commented Aug 5, 2015

Ok, I ran through it, applied some changes and waiting for others to comment ...
Before merging, a couple of things still are needed ...

@CoinGame @brossi

  • from daemon how can I list my receive NSR addresses?

This comment has been minimized.

Show comment Hide comment
@brossi

brossi Aug 5, 2015

Contributor

listreceivedbyaddress

listreceivedbyaddress [minconf=1] [includeempty=false]

[minconf] is the minimum number of confirmations before payments are included. Run against an individual unit (so you'll need to specify whether or not you want this to be for NBT or NSR address with the -unit=B or -unit=S command).

Contributor

brossi commented Aug 5, 2015

listreceivedbyaddress

listreceivedbyaddress [minconf=1] [includeempty=false]

[minconf] is the minimum number of confirmations before payments are included. Run against an individual unit (so you'll need to specify whether or not you want this to be for NBT or NSR address with the -unit=B or -unit=S command).

This comment has been minimized.

Show comment Hide comment
@desrever-nu

desrever-nu Aug 5, 2015

Contributor

Thanks !

Contributor

desrever-nu commented Aug 5, 2015

Thanks !

This comment has been minimized.

Show comment Hide comment
@desrever-nu

desrever-nu Aug 6, 2015

Contributor

I have not received additional feedback, I think we can merge @brossi @CoinGame

Contributor

desrever-nu commented Aug 6, 2015

I have not received additional feedback, I think we can merge @brossi @CoinGame

CoinGame added a commit that referenced this pull request Aug 6, 2015

@CoinGame CoinGame merged commit 91d1b39 into NuNetwork:gh-pages Aug 6, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment