Skip to content

Conversation

@barrystyle
Copy link

To complement the Dandelion implementation; prevent signalling RBF on transactions generated from the UI. Other functions will remove unaffected (a soft-disable).

Copy link

@SmartArray SmartArray left a comment

Choose a reason for hiding this comment

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

utACK.

This corresponds to TODO-002 from the list that I composed here: #63 (comment)

Copy link

@fgabelmannjr fgabelmannjr left a comment

Choose a reason for hiding this comment

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

utACK

Copy link
Member

@gto90 gto90 left a comment

Choose a reason for hiding this comment

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

utACK

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.

ACK. Compiles, runs, sends, GUI shows it removed.

However during the process I noticed we have some theming issues. The icons in boxes to the right of Pay To box are not showing on send page like they should, but this issue was already there.

Before PR:

Screen Shot 2022-01-28 at 8 05 14 AM

After PR:

Screen Shot 2022-01-28 at 7 55 43 AM

@gto90 gto90 merged commit ad5a592 into DigiByte-Core:feature/8.22.0 Jan 30, 2022
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