Bitcoin Forum
November 05, 2024, 11:12:05 AM *
News: Latest Bitcoin Core release: 28.0 [Torrent]
 
   Home   Help Search Login Register More  
Pages: [1]
  Print  
Author Topic: Bounty for CPFP Enabled Wallet  (Read 2563 times)
d4n13 (OP)
Full Member
***
Offline Offline

Activity: 210
Merit: 104


“Create Your Decentralized Life”


View Profile
July 15, 2015, 08:12:27 AM
Last edit: July 21, 2015, 02:59:53 PM by d4n13
 #1

CPFP (Child Pays for Parent) Bounty
3FEUByMeaxrNmBCVYjvsnhyAjiUdat5i7M

Anyone who can put a CPFP transaction creation feature into any wallet featured on the bitcoin homepage can claim this bounty.  

I outlined the basic idea of the bounty in this BIP proposal before I was educated on CPFP.

To claim the bounty / pizza you will need to convince the following holders of the multisig

Rules:
  • Must be a wallet featured on the bitcoin homepage
  • Must be either accepted into primary build branch of (1), or available in an official release / beta / dev build of (1)

References:

Andreas Schildbach
Hero Member
*****
Offline Offline

Activity: 483
Merit: 551


View Profile
July 15, 2015, 09:41:53 AM
 #2

We're already working on getting child-pays-for-parent into Bitcoin Wallet. The branch is here: https://github.com/schildbach/bitcoin-wallet/commits/fee-cpfp

It's still a bit rough and needs work. The biggest missing part is PIN entry, so you currently need to unset your PIN before raising the fee.

Any code review is appreciated.
d4n13 (OP)
Full Member
***
Offline Offline

Activity: 210
Merit: 104


“Create Your Decentralized Life”


View Profile
July 15, 2015, 03:09:42 PM
Last edit: July 15, 2015, 03:34:43 PM by d4n13
 #3

We're already working on getting child-pays-for-parent into Bitcoin Wallet. The branch is here: https://github.com/schildbach/bitcoin-wallet/commits/fee-cpfp

It's still a bit rough and needs work. The biggest missing part is PIN entry, so you currently need to unset your PIN before raising the fee.

Any code review is appreciated.
Wow... fantastic news.... I'll be happy to look at it...

Setting up an android build server will take a bit.  If no one beats me too it, I'll try to post a testnet build from the branch... expect it to take a while.

achow101
Moderator
Legendary
*
expert
Offline Offline

Activity: 3542
Merit: 6885


Just writing some code


View Profile WWW
July 15, 2015, 03:40:58 PM
 #4

Does the CPFP patch have to make it into the stable build released by the developers of the wallets or can it just be a patch that you apply yourself?

d4n13 (OP)
Full Member
***
Offline Offline

Activity: 210
Merit: 104


“Create Your Decentralized Life”


View Profile
July 15, 2015, 03:49:48 PM
 #5

Does the CPFP patch have to make it into the stable build released by the developers of the wallets or can it just be a patch that you apply yourself?
I added a rules section to OP
Quote
Rules:
  • Must be a wallet featured on the bitcoin homepage
  • Must be either accepted into primary build branch of (1), or available in a production / beta / dev build of (1)

Andreas Schildbach
Hero Member
*****
Offline Offline

Activity: 483
Merit: 551


View Profile
July 16, 2015, 03:06:38 PM
 #6

I just updated our branch so that it supports encrypted wallets (spending PIN).
Andreas Schildbach
Hero Member
*****
Offline Offline

Activity: 483
Merit: 551


View Profile
July 17, 2015, 03:18:50 PM
 #7

The feature is now finished, merged and starts to roll out to 10% of users.

If you feel like skipping the queue you can download directly from here:

https://github.com/schildbach/bitcoin-wallet/releases/tag/v4.37
d4n13 (OP)
Full Member
***
Offline Offline

Activity: 210
Merit: 104


“Create Your Decentralized Life”


View Profile
July 17, 2015, 04:16:24 PM
 #8

The feature is now finished, merged and starts to roll out to 10% of users.

If you feel like skipping the queue you can download directly from here:

https://github.com/schildbach/bitcoin-wallet/releases/tag/v4.37
All over it... Thx... I assume the "test" is testnet.

d4n13 (OP)
Full Member
***
Offline Offline

Activity: 210
Merit: 104


“Create Your Decentralized Life”


View Profile
July 21, 2015, 03:02:30 PM
 #9

The feature is now finished, merged and starts to roll out to 10% of users.

If you feel like skipping the queue you can download directly from here:

https://github.com/schildbach/bitcoin-wallet/releases/tag/v4.37
Awarded
to Andreas Schildbach

congrats!
https://blockchain.info/tx/8ec943be37e8e45ba60e0c3ae82323cade5e084b25e9154714e86a5d80a1a44b

unamis76
Legendary
*
Offline Offline

Activity: 1512
Merit: 1012


View Profile
July 21, 2015, 03:24:48 PM
 #10

Good news that this has been implemented... However I don't use the software in which it was Cheesy Hoping this at least makes it to the Core wallet.
arnuschky
Hero Member
*****
Offline Offline

Activity: 518
Merit: 502


View Profile
July 26, 2015, 10:15:18 PM
 #11


A bit OT, but which other nodes/wallets implement CPFP already out-of-the-box?
brianddk
Jr. Member
*
Offline Offline

Activity: 47
Merit: 16


View Profile WWW
July 26, 2015, 10:49:44 PM
 #12


A bit OT, but which other nodes/wallets implement CPFP already out-of-the-box?
Only Android Bitcoin-Wallet...

No others that I've seen, which is why it won the bounty.
voisine
Member
**
Offline Offline

Activity: 115
Merit: 19


View Profile
August 26, 2015, 03:26:16 PM
 #13

We included CPFP in breadwallet a few months back (two months before the bounty was created) for when the user spends unconfirmed non-change inputs. Happy to let the bounty go to Andreas though. Andreas and co. are doing good work.

https://github.com/voisine/breadwallet/blob/v0.5.1/BreadWallet/BRWallet.m#L382
unamis76
Legendary
*
Offline Offline

Activity: 1512
Merit: 1012


View Profile
August 26, 2015, 03:43:30 PM
 #14

We included CPFP in breadwallet a few months back (two months before the bounty was created) for when the user spends unconfirmed non-change inputs. Happy to let the bounty go to Andreas though. Andreas and co. are doing good work.

https://github.com/voisine/breadwallet/blob/v0.5.1/BreadWallet/BRWallet.m#L382

Wasn't aware that one of my favorite wallets already had it Smiley Nice job. It will probably come in handy some day Grin
Pages: [1]
  Print  
 
Jump to:  

Powered by MySQL Powered by PHP Powered by SMF 1.1.19 | SMF © 2006-2009, Simple Machines Valid XHTML 1.0! Valid CSS!