Skip to content

Conversation

@UdjinM6
Copy link

@UdjinM6 UdjinM6 commented Apr 25, 2023

Issue being fixed or feature implemented

we failed to backport 13216 correctly in #4359

noticed this while reviewing/testing #5255

What was done?

fix it

How Has This Been Tested?

run qt with -resetguisetting and check info with and without the patch on testnet for example (or tweak regtest params and test there)

Breaking Changes

n/a

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation
  • I have assigned this pull request to a milestone (for repository code-owners and collaborators only)

failed to backport 13216 correctly in 4359
@UdjinM6 UdjinM6 added this to the 20 milestone Apr 25, 2023
Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

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

utACK for squash merge

@PastaPastaPasta PastaPastaPasta merged commit 6c5d6c1 into dashpay:develop Apr 26, 2023
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request May 22, 2023
## Issue being fixed or feature implemented
we failed to backport 13216 correctly in dashpay#4359

noticed this while reviewing/testing dashpay#5255 

## What was done?
fix it

## How Has This Been Tested?
run qt with `-resetguisetting` and check info with and without the patch
on testnet for example (or tweak regtest params and test there)

## Breaking Changes
n/a

## Checklist:
- [x] I have performed a self-review of my own code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have added or updated relevant unit/integration/functional/e2e
tests
- [ ] I have made corresponding changes to the documentation
- [x] I have assigned this pull request to a milestone _(for repository
code-owners and collaborators only)_
@UdjinM6 UdjinM6 modified the milestones: 20, 19.1 May 22, 2023
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.

2 participants