Add toString() methods for regularly logged entities

Question

Add toString() methods for regularly logged entities

Closed this issue 9 months ago · 7 comments

Brutus5000 commented 9 months ago

User -> print id and username
Game -> print id and title

Answer 1 · 2024-01-11T15:26:09.000Z

I would like to contribute :)

Answer 2 · 2024-01-11T21:55:26.000Z

Well then go for it!

Answer 3 · 2024-01-12T12:22:35.000Z

Hi @Brutus5000, Could you please assign this issue to me?
So, I can get started with toString() methods for the requested classes.

Answer 4 · 2024-01-13T20:29:53.000Z

@Brutus5000 Just wanted to confirm if I need to override toString() methods for com.faforever.api.data.domain.User and com.faforever.api.data.domain.Game only.

Answer 5 · 2024-01-13T20:49:51.000Z

Correct. Also check out to use Lombok's @tostring

Answer 6 · 2024-01-13T20:50:57.000Z

Sure and thanks for the clarification.

Answer 7 · 2024-01-13T22:22:44.000Z

@Brutus5000 I've created a pull request for this issue. Kindly review it once you get time.