-
xmr-pr
xiphon opened pull request #2991: cmake: implement MacOS 'release' build + CI support
-
xmr-pr
-
slstsrv
.merges
-
xmr-pr
#2940 #2948 #2949 #2974 #2976 #2977 #2978
-
slstsrv
.merge+ #2991
-
xmr-pr
Added
-
xmr-pr
rating89us opened issue #2993: History: amounts like 0.000000000001 don't display correctly
-
xmr-pr
-
xmr-pr
rating89us opened pull request #2992: history: add receiving address label + fix search and amount display
-
xmr-pr
-
xmr-pr
glv2 opened pull request #2994: Mining: Indicate probability of finding a block in one day
-
xmr-pr
-
xmr-pr
rating89us opened pull request #2995: history: change sent transaction dot color from orange to red
-
xmr-pr
-
xmr-pr
rating89us opened pull request #2997: history: fix amount display + remove trailing zeros in amount
-
xmr-pr
-
xmr-pr
rating89us opened pull request #2996: history: fix search filter not working when there is a pending tx
-
xmr-pr
-
dsc_
.merge+ #2997 #2996 #2988 #2978 #2976 #2949
-
dsc_
.merges
-
xmr-pr
#2940 #2948 #2949 #2974 #2976 #2977 #2978 #2988 #2991 #2996 #2997
-
dsc_
.merge+ #2935 #2915
-
xmr-pr
Added
-
dsc_
does `ratingus85` ever come here?
-
dsc_
he seems to redesign everything, might be nice to discuss it beforehand first
-
dsc_
¯\_(ツ)_/¯
-
luigi1111w
89
-
moneromooo
Would that be close enough ?
-
moneromooo
Maybe it was that person's age when they registered in 2016 ^_^
-
dsc_
Missed my window >:|
-
dsc_
.merge+ #2994
-
xmr-pr
Added
-
dsc_
luigi1111w: can haz?
-
luigi1111w
yes
-
luigi1111w
what was the question
-
dsc_
merge-wave when possible :)
-
luigi1111w
but they've piled up and look big now
-
dsc_
should have used the opportunity of you saying Yes to a future unspecified question by asking for core team membership ;x
-
dsc_
Maybe do half?
-
dsc_
.... a quarter? :D
-
» dsc_ is working on another Monero GUI, where he can merge and admin the issue/PR tracker himself
-
» dsc_ can't wait
-
dsc_
luigi1111w: thank you sir
-
luigi1111w
hey I did a quarter nice
-
dsc_
o/
-
dEBRUYNE
dsc_: Do you disagree with some of the redesigns?
-
dsc_
dEBRUYNE: He's doing rather great work honestly
-
dsc_
just some small tweaks and/or suggestions that could be discussed over IRC, especially before coding them could help
-
dsc_
Previously i've discussed features/redesign with other people first before coding large PRs
-
dsc_
dEBRUYNE: what do you think of this one:
monero-project/monero-gui #2936 ?
-
slstsrv
fwiw the sending screen redesign is from knueffelbund designs
-
slstsrv
but yep 2936 can be discussied, it encourages using a fresh address everytime which is good for privacy but might be a bit more complicated to use
-
dEBRUYNE
Haven't looked at that one in detail yet to be honest
-
dEBRUYNE
<dsc_> dEBRUYNE: He's doing rather great work honestly <= Ok great!
-
dsc_
slstsrv: I'm running that PR and it's pretty great, I ought to review it properly
-
dsc_
The "always new address" works well, I'd even argue the UX proposed here is less complicated
-
dsc_
It marks the address as "used" as soon as you copy it to the clipboard
-
dsc_
just wondering if it automatically increments the lookahead
-
dsc_
in addition, lookaheads are stored inside wallet cache, so a mnemonic seed restore on another pc would require you to also correctly specify that, else you might end up reusing old subaddys?
-
slstsrv
do you know the default lookahead?
-
moneromooo
Either 200/50 or 50/200.
-
dsc_
200 or 500 from the top of my head
-
slstsrv
afaik it is less when using ledger / trezor which could be a problem
-
moneromooo
Ledger is somehting like 10 I believe.
-
slstsrv
right, using 10 addresses without receiving can quickly happen if it always generates a new one
-
dsc_
So if the GUI is generous in giving out addresses, restoring might become a hurdle
-
dsc_
(if we don't have the proper UX to signal that lookaheads are important)
-
dsc_
Suppose the GUI could loop to find out which subaddresses have been used previously (incoming txs) till it finds a clean one
-
slstsrv
right but we don't want to do that else we might give out the same subaddress to multiple people
-
dsc_
true...
-
dsc_
I guess this is a showstopper for this PR then? Anyone got a good idea? :D
-
dsc_
New seed format: include restore height and lookahead :)
-
selsta
lol
-
dsc_
I don't hear any No's :)
-
dsc_
Also another thing I'm wondering; if the GUI hands out addresses like there is no tomorow ... what is the wallet scanning performance like after years of usage?
-
dsc_
Or is this a moot point because It's very easy to just replace your wallet for a new one when it gets slow?
-
dsc_
questions .. questions
-
selsta
should be negligible
-
selsta
also it only generates a new subaddress if you copy the address on the receive screen which should not happen that often
-
selsta
but the lookahead on ledger / trezor is a valid concern
-
selsta
21:48 <selsta> should be negligible <-- subaddresses are designed in a way to not affect scanning performance, only initially creating the lookup table (using lookahead) is resource intensive
-
dsc_
ah okay
-
selsta
.merge+ #2989
-
xmr-pr
Added
-
selsta
.merge+ #2935
-
xmr-pr
selsta opened pull request #2998: workflows: fix macOS build
-
xmr-pr
-
dsc_
.merge+ #2998
-
xmr-pr
Added
-
dsc_
.merges
-
xmr-pr
#2915 #2935 #2989 #2991 #2998
-
selsta
xiphon: do you have idea about my issue with #2991 when using non brew Qt or should we merge this for now?
-
selsta
an*
-
xiphon
selsta: i need an environment to reproduce the issue and fix it
-
selsta
ok, will see how to do this
-
xiphon
how do you install qt? Downloading .dmg and running some installation script?
-
selsta
using normal Qt installer
-
xiphon
Could this be done from a terminal?
-
selsta
-
selsta
Qt online installer
-
selsta
will see
-
selsta
else we can purchase a Mac VM for a month
-
xiphon
"should we merge this for now?" -> yes, the PR implements CMake reelase build on MacOS with qt installed through package manager
-
selsta
sounds good, the issue I have is not related to that PR anyway
-
selsta
though it failed to build now with some lrelease related issue
-
xmr-pr
xmrdsc opened pull request #2999: Donate to Monero feature
-
xmr-pr
-
kinghat[m]
<selsta "else we can purchase a Mac VM fo"> there is a mac vm snap you can use called sosumi or something like that
-
kinghat[m]
-
xiphon
"though it failed to build now with some lrelease related issue" -> due to merged CMake translations PR, wokring on it
-
selsta
kinghat[m]: thanks, though not sure if xiphon wants to bother installing this on his system, a VM would be easier
-
selsta
running a Mac VM on Linux / Windows is quite hacky
-
kinghat[m]
ya its all hacky.
-
selsta
a Mac VPS would be easier*
-
kinghat[m]
that looks like an easy way to go about it though
-
kinghat[m]
mac vps isnt free i dont think
-
kinghat[m]
unless ppl are handing them out to oss projects
-
selsta
yea I know but I’m sure we could get one funded for a month :P
-
kinghat[m]
fuck apple
-
kinghat[m]
pardon my english
-
selsta
they sure make it difficult for open source projects
-
kinghat[m]
they dont have a problem using oss
-
kinghat[m]
i heard that they didnt like the gpl3 of samba so they rolled their own or something
-
selsta
.merges
-
xmr-pr
#2915 #2935 #2989 #2991 #2998