💬 hebasto commented on issue "Generalized fee bumping":
(https://github.com/bitcoin-core/gui/issues/822#issuecomment-2227992916)
cc @achow101 @Sjors @GBKS

💬 Sjors commented on pull request "Have createNewBlock() return a BlockTemplate interface":
(https://github.com/bitcoin/bitcoin/pull/30440#issuecomment-2228006803)
Pushed a commit that adds `submitSolution` to the `BlockTemplate` interface.

✅ fanquake closed an issue: "Mini miner `AncestorFeerateComparator` Signed Integer Overflow"
(https://github.com/bitcoin/bitcoin/issues/30284)

✅ fanquake closed an issue: "fuzz: mini_miner_selection: ASSERT: mock_template_txids.size() <= blocktemplate->block.vtx.size()"
(https://github.com/bitcoin/bitcoin/issues/30367)

🚀 fanquake merged a pull request: "MiniMiner: use FeeFrac in AncestorFeerateComparator"
(https://github.com/bitcoin/bitcoin/pull/30412)

👍 hebasto approved a pull request: "Keep focus on "Hide" while ModalOverlay is visible"
(https://github.com/bitcoin-core/gui/pull/795#pullrequestreview-2177280212)
re-ACK 992b1bbd5da95ee782515fb0f5674bb7a02684b2

✅ hebasto closed an issue: "Weird focus rect displayed on inital sync"
(https://github.com/bitcoin-core/gui/issues/783)

🚀 hebasto merged a pull request: "Keep focus on "Hide" while ModalOverlay is visible"
(https://github.com/bitcoin-core/gui/pull/795)

👍 dergoegge approved a pull request: "kernel: De-globalize static validation variables"
(https://github.com/bitcoin/bitcoin/pull/30425#pullrequestreview-2177300151)
Code review ACK 51fa26239af9bbfd44029aaf595cb4c6a8d4a75d

💬 hebasto commented on issue "Welcome screen uses GiB values as GB, risking running out of space":
(https://github.com/bitcoin-core/gui/issues/821#issuecomment-2228117027)
Also see https://github.com/bitcoin/bitcoin/pull/29678.

👍 dergoegge approved a pull request: "fuzz: bound some miniscript operations to avoid fuzz timeouts"
(https://github.com/bitcoin/bitcoin/pull/30197#pullrequestreview-2177308717)
utACK bc34bc288824978ef4b98e8802b47cb863c8a8c2

👍 dergoegge approved a pull request: "fuzz: fix key size in `crypter`"
(https://github.com/bitcoin/bitcoin/pull/30373#pullrequestreview-2177316284)
utACK 4383dc90bac1b5def73352fe222f99807d8ca4dd

💬 glozow commented on pull request "Ephemeral Dust":
(https://github.com/bitcoin/bitcoin/pull/30239#issuecomment-2228142149)
concept ACK

👍 dergoegge approved a pull request: "Add fuzz test for FSChaCha20Poly1305, AEADChacha20Poly1305"
(https://github.com/bitcoin/bitcoin/pull/28263#pullrequestreview-2177332281)
tACK 39805638663bdd567a50ecac962631b45ce8d73a

💬 dergoegge commented on pull request "Add fuzz test for FSChaCha20Poly1305, AEADChacha20Poly1305":
(https://github.com/bitcoin/bitcoin/pull/28263#discussion_r1677597969)
nit:

```suggestion
auto aad{rng.randbytes<std::byte>(aad_length)};
```

💬 Sjors commented on pull request "Have createNewBlock() return a BlockTemplate interface":
(https://github.com/bitcoin/bitcoin/pull/30440#issuecomment-2228153853)
Also added `getHashPrevBlock()`.

💬 hebasto commented on pull request "OptionsDialog: Allow Maximize of window":
(https://github.com/bitcoin-core/gui/pull/826#issuecomment-2228155212)
The PR description is empty, so I have to ask: why is this needed?

🤔 hebasto reviewed a pull request: "OptionsDialog: Prefer to stretch actual options area rather than waste space"
(https://github.com/bitcoin-core/gui/pull/827#pullrequestreview-2177355412)
Tested b71bfd9eef8b0017cef3c05361c02ddbd837e6ac on Ubuntu 24.04 + Qt 5.15.13:

- the master branch @ 6ae903e24afa95214e59a9dc87dc74db3535a5b5:

- this PR:

@GBKS What do you think?

💬 GBKS commented on pull request "OptionsDialog: Prefer to stretch actual options area rather than waste space":
(https://github.com/bitcoin-core/gui/pull/827#issuecomment-2228175727)
That is a nice improvement.

💬 1440000bytes commented on issue "scriptPubKey no address":
(https://github.com/bitcoin/bitcoin/issues/30450#issuecomment-2228177423)
https://github.com/bitcoin/bitcoin/issues/22524#issuecomment-884663685