rcx icon indicating copy to clipboard operation
rcx copied to clipboard

switch logging to json and remove clunky notificationbuilding

Open newhinton opened this issue 3 years ago • 4 comments

Signed-off-by: Felix Nüsse [email protected]

~~There is a rather important todo open:~~ ~~I have no code for translating bytes to a human readable form. The stub is there, but it just adds 'bytes' to the value.~~

I can also add translations for the notification.

newhinton avatar Jun 28 '21 17:06 newhinton

Added a few comments; see inline.

x0b avatar Jul 04 '21 20:07 x0b

Apart from the superficial comments, this is basically ready for merge.

x0b avatar Jul 09 '21 13:07 x0b

I have added the changes. Any news on the other pr? I have ideas to further change the notifications and other stuff in rcx, but i want to wait on the major changes beeing merged before i start working on further features

newhinton avatar Jul 22 '21 13:07 newhinton

I also updated the notification, so that it looks better and not as technical as before

newhinton avatar Sep 03 '21 09:09 newhinton