namecoin / namecoin-legacy

Legacy client. New version here: https://github.com/namecoin/namecoin-core Note the release branch! - Official website:
https://namecoin.org
MIT License
448 stars 177 forks source link

Default transaction fee #167

Closed phelixbtc closed 10 years ago

phelixbtc commented 10 years ago

Don't create free TXs that can get stuck by default.

domob1812 commented 10 years ago

I'm fine with this - ACK. Is it intentional that you only change it for the Qt and not the RPC calls? But I think this may be a good idea - users of the RPC interface can be expected to know what they are doing. Also it is good that this only changes the creation rules and not the tx acceptance / relaying rules. (For these, I think that we should stick to free transactions - at least until we have discussed the issue in more depth.)

phelixbtc commented 10 years ago

The change in main.cpp should also change the default for RPC calls. Did I miss something? Users who know what they doing can still lower the fee in the GUI or add a line to namecoin.conf and can take action if a transaction should not go through.

domob1812 commented 10 years ago

Ah yes, stupid me. Merging.