-
ErCiccione[m]
.merge+ #994
-
xmr-pr
Added
-
ErCiccione[m]
-
xmr-pr
Sunray-Nucleon opened issue #995: Replace Link in RandomX Moneropedia 'an article dedicated to RandomX'
-
xmr-pr
-
fluffypony
ErCiccione[m]: did you ever see the old 404 page?
-
fluffypony
I resurrected it for your enjoyment
-
fluffypony
-
ErCiccione[m]
As far as i remember no, I took a look at the code locally and looked fun
-
ErCiccione[m]
Cool!
-
ErCiccione[m]
If it wasn't js stuff, would have been nice to keep it
-
fluffypony
if the JS doesn't load it just shows the world and dog statically
-
fluffypony
could probably redo it with CSS animations nowadays
-
ErCiccione[m]
yeah, but without balloon. Maybe we could make something nicer in the future, my version is super minimalistic.
-
ErCiccione[m]
I pushed updates to the new homepage according to the feedbacks received:
monero-project/monero-site #948
-
ErCiccione[m]
would be good to have some final opinions and reviews, so that we can merge it
-
binaryFate
-
ErCiccione[m]
Ok. Thanks for the ping
-
ErCiccione[m]
Could somebody give a quick review to 992 and 996?
-
ErCiccione[m]
We should have them merged together with the blog post
-
ErCiccione[m]
.merge+ #996
-
xmr-pr
Added
-
binaryFate
ErCiccione[m]: I'd be glad to go partially offline in an hour or so, do you want me to update site before that?
-
ErCiccione[m]
The PR with the blog post has no reviews, yet. There is nothing to update yet
-
binaryFate
Several people said it looked ok on IRC
-
ErCiccione[m]
we need 1000 992 and 996 reviewed and merged together
-
ErCiccione[m]
blog post, one link fix and reset the common issues
-
binaryFate
Ah ok, sorry I forgot about other stuff besides blog post
-
ErCiccione[m]
if you want to go ahead and merge them i'm ok with that. The blog post was reviewed on IRC and the others 2 is trivial changes
-
binaryFate
I think the blog post should be out, there is no reason to wait. The other stuff might wait potentially, or just follow usual pace/process
-
binaryFate
up to you
-
ErCiccione[m]
but since it's trivial stuff people could also review quickly. hint hint
-
ErCiccione[m]
992 should go with the blog post. Or people will think the CLI has issues that it actually doesn't have
-
binaryFate
lemme have a look...
-
ErCiccione[m]
the fix could wait
-
binaryFate
links are good anyway
-
ErCiccione[m]
oh yes those were already reviewed
-
binaryFate
so what's missing
-
ErCiccione[m]
That's it it's those 3
-
binaryFate
Sorry, I mean what's missing to merge them?
-
binaryFate
I don't want to overstep the processes, I'm just unclear what we're waiting for in practice
-
binaryFate
I can only update site soon or not until tomorrow, and now that release is out and everyone will talk about it, would be good that people can tweet post on website rather than external sources
-
ErCiccione[m]
ah ok got it. Now that you reviewed them, ere only need luigi to merge them if he is online
-
ErCiccione[m]
luigi1111^
-
luigi1111
merge what
-
ErCiccione[m]
992 996 and 1000
-
binaryFate
luigi1111 can you do it soon? You want me to do? (it's the blog post + 2 minor stuff reviewed)
-
binaryFate
Cause I can update the website soon or not till tomorrow
-
luigi1111
go ahead. I need to go get 2fa codes. away for the weekend
-
binaryFate
allrighty
-
binaryFate
ErCiccione[m]: it's all merged and deployed
-
ErCiccione[m]
Great, thanks :)