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

Bugfix: fixes/2078 for the uiux-revamp branch (mitigating taproot issue) #2088

Merged
merged 5 commits into from
Jan 30, 2023

Conversation

k9ert
Copy link
Collaborator

@k9ert k9ert commented Jan 27, 2023

applies #2087 to uiux-revamp. No special changes for v2 needed!
original bug: #2078

@netlify
Copy link

netlify bot commented Jan 27, 2023

Deploy Preview for specter-desktop-docs canceled.

Name Link
🔨 Latest commit 5454533
🔍 Latest deploy log https://app.netlify.com/sites/specter-desktop-docs/deploys/63d3c92b4cbdb600085546f9

@k9ert k9ert added this to the UIUX revamp (Specter 2.0) milestone Jan 27, 2023
@k9ert k9ert added the v2 label Jan 27, 2023
* Some smaller improvements and fixes

* mitigating 2078

* removed Microsoft link tracking

* wrap up a failed update properly

* better decode and better error_handling
@k9ert k9ert changed the base branch from master to uiux-revamp January 27, 2023 12:53
@k9ert k9ert changed the title Uiux/fixes/2078 Bugfix: fixes/2078 for the uiux-revamp branch (mitigating taproot issue) Jan 30, 2023
super_dict["utxo_amount"] = self.utxo_amount
super_dict["ismine"] = (self["ismine"] or self.ismine,)
return super_dict
# def __dict__(self):
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why is this commented out? If not needed, I'd rather delete.

Copy link
Collaborator

@moneymanolis moneymanolis left a comment

Choose a reason for hiding this comment

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

Approved, just see one little comment.

@moneymanolis moneymanolis merged commit f2d0857 into cryptoadvance:uiux-revamp Jan 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants