- Issue created by @danielspeicher
- @danielspeicher opened merge request.
- Status changed to Needs review
over 1 year ago 3:04pm 1 June 2023 - 🇩🇪Germany danielspeicher Steisslingen
I also did some code cleanups, e.g. add comments and avoid static calls.
I will also open a new ticket for the rest of the cleanups
- Status changed to Needs work
over 1 year ago 4:43pm 5 June 2023 - 🇩🇪Germany jurgenhaas Gottmadingen
I have reviewed this, it's a bit unfortunate that so many off-topic changes are in there. I guess the original purpose of this issue is only in 2 changes of the MR, so I hope I haven't missed anything important.
- Status changed to Needs review
over 1 year ago 12:28pm 7 June 2023 - 🇩🇪Germany danielspeicher Steisslingen
Thanks for the review. Yes, next time an extra ticket will be opened.
- Status changed to Needs work
over 1 year ago 1:46pm 7 June 2023 - 🇩🇪Germany jurgenhaas Gottmadingen
Looking great already. There is only that user load issue left, which could also be handled in a separate issue, if you prefered.
- Status changed to RTBC
over 1 year ago 1:53pm 7 June 2023 - 🇩🇪Germany danielspeicher Steisslingen
Ah, I now understand what you mean. I will open another ticket for this.
-
danielspeicher →
committed 1e90de1a on 1.1.x
Issue #3363759 by danielspeicher, jurgenhaas: Enhance download of ticket...
-
danielspeicher →
committed 1e90de1a on 1.1.x
- Status changed to Fixed
over 1 year ago 1:59pm 7 June 2023 Automatically closed - issue fixed for 2 weeks with no activity.