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

Set one kin = 1000000 units ==> 5 decimal points#16

Merged
oryband merged 1 commit intokinecosystem/masterfrom
kin_native
Nov 13, 2018
Merged

Set one kin = 1000000 units ==> 5 decimal points#16
oryband merged 1 commit intokinecosystem/masterfrom
kin_native

Conversation

@avi-kik
Copy link
Copy Markdown

@avi-kik avi-kik commented Nov 5, 2018

No description provided.

@avi-kik avi-kik requested a review from gadisr November 5, 2018 08:36
@avi-kik
Copy link
Copy Markdown
Author

avi-kik commented Nov 5, 2018

@gadisr,

The change is completely untested. I didn't even build. Please do so and run through the tests. I have no way of figuring out which unit tests may fail because of this change, and I don't know if the change will even have the desired effect.

@oryband oryband force-pushed the kin_native branch 3 times, most recently from 64ab485 to 67d1467 Compare November 13, 2018 15:11
@oryband oryband changed the title Set one kin = 10000 units. Set one kin = 1000000 units ==> 5 decimal points Nov 13, 2018
@oryband
Copy link
Copy Markdown

oryband commented Nov 13, 2018

FYI: will update tests where necessary in a future branch

@oryband oryband requested review from fuzzyami and ordemri November 13, 2018 15:12
- instead of 7 (10000000) previously
- this allows for a single account to hold 10 trillion native tokens
@oryband oryband merged commit ecec182 into kinecosystem/master Nov 13, 2018
@oryband oryband deleted the kin_native branch November 13, 2018 15:14
@avi-kik
Copy link
Copy Markdown
Author

avi-kik commented Nov 13, 2018

My first comment was correct. You changed the spec without communicating to anyone who is actually affected.

oryband added a commit that referenced this pull request May 2, 2019
this will allow to use Horizon v0.16.1 on Ecosystem network

Revert "Merge pull request #16 from kinecosystem/kin_native"

This reverts commit ecec182, reversing
changes made to 407b4f4.
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.

2 participants