Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename Satoshi to DigiSatoshi in QT #229

Merged
merged 3 commits into from
May 21, 2024

Conversation

saltedlolly
Copy link

Clearly renaming it to a DigiBit was proving controversial. This change now makes it match the current "official" name in the DgbWiki: https://dgbwiki.com/index.php?title=DigiByte#Subunits

I elected to leave the abbreviation as 'sat' rather than 'dSat' for now. I think we should either update the wiki to use the abbreviation 'sat', or update DigiByte Core to use dSat in another PR. That way the term is consistent.

This matches the currently accepted "official" name.
ghost
ghost previously approved these changes Apr 4, 2024
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Code is working. Tnx @saltedlolly
Screenshot from 2024-04-04 17-37-53

@JaredTate
Copy link

I see no harm in making it unique to DGB at all. Would "DigiToshi" be better because it's shorter? Just spitballing ideas out loud.

@saltedlolly
Copy link
Author

My main objective for this change is simply to make the term consistent everywhere it is used, as it has been called different things in different places. Since there was a lot of opposition to DigiBit, I thought maybe I should keep it simple and just rename it to match the name on the wiki. At least then it is consistent. @mctrivia and others pointed out that abbreviation "sat" is used fairly universally, which is a fair point, even if personally I would prefer a unique name. Calling it DigiSatoshi does mean that abbreviation fits, even if it's a bit of mouthful. If we call it DigiToshi it's not such a good fit for the abbreviation "sat". Just my 2c.

(Another option is "DigiSat" as I suggested on the Issue thread which is a bit shorter and fits the abbreviation, but seeing as they just get called "sats" anyway, I figured the full name of "DigiSatoshi" will do.)

@mctrivia
Copy link

mctrivia commented Apr 5, 2024

Ah but the wiki has been proposed to be changed to fit everything else. Sat.

You definetly will fail in trying to standardise to anything but sat because all the 3rd party applications call sat.

@saltedlolly
Copy link
Author

saltedlolly commented Apr 5, 2024

Ah but the wiki has been proposed to be changed to fit everything else. Sat.

You definetly will fail in trying to standardise to anything but sat because all the 3rd party applications call sat.

Which is why I left it as "sat".

That said Dgbwiki currently says they are called dSats or Digis, so if we are calling them "Sats" it should be updated to reflect that. Once this PR is merged I will do that so the name is consistent everywhere.

Copy link

@JaredTate JaredTate left a comment

Choose a reason for hiding this comment

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

cACK. Not sure where we are on this, but I am not opposed to any renaming of "Sat". I am ok with this.

@gto90
Copy link
Member

gto90 commented May 21, 2024

cACK

Copy link
Member

@ycagel ycagel left a comment

Choose a reason for hiding this comment

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

cACK

@ycagel ycagel merged commit 22528bf into DigiByte-Core:develop May 21, 2024
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.

5 participants