Skip to content

Make the minting table's address display match the transaction table#459

Open
IngCr3at1on wants to merge 2 commits intoPaycoinFoundation:masterfrom
IngCr3at1on:label-for-minting-tab
Open

Make the minting table's address display match the transaction table#459
IngCr3at1on wants to merge 2 commits intoPaycoinFoundation:masterfrom
IngCr3at1on:label-for-minting-tab

Conversation

@IngCr3at1on
Copy link
Contributor

puts the addresses in parenthesis and puts the label before it.

this resolves #305

puts the addresses in parenthesis and puts the label before it.

this resolves #305
@IngCr3at1on
Copy link
Contributor Author

.75 hours

@MitchellCash
Copy link
Contributor

As discussed in chat and to just make the comment public for discussion, I would prefer to call the already existing MintingTableModel::lookupAddress to make this work.

@IngCr3at1on
Copy link
Contributor Author

@MitchellCash yeah I'll fix that, I didn't notice the function in there as it's dead currently lol.

@IngCr3at1on
Copy link
Contributor Author

@MitchellCash this has been changed per our discussion (and retested) I'll rebase it after review (before merge). (20 minutes)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add label display for address in minting tab

2 participants