Feedback from Simon

[otobrglez]

Feedback from @szlend:

Ej to zgleda kar proper za prvi rust projekt 😄
Par stvari:

• če rabiš nek function parameter samo za branje, je convention da ga passaš kot reference &. (npr. anchor.post_login(&credentials, &token)). Drugače rust naredi move, kar pomeni da po klicu funkcije ne bo več accessible. Izjema so tipi ki implementirajo Copy, npr. u32. Al pa če bi funkcija iz nekega razloga mogla klonirat parameter (pol se lahko caller odloči a hoče clone ali move).
• isto velja za funkcije ki sprejemajo String, tam lahko uporabiš tip &str (če je foo: String pol &foo downcasta v tip &str). To je itak good practice ker pol lahko passaš tudi string literalle in jih ni treba convertat z "neke".to_string() . Če te zanima kako ta downcasting deluje, si lahko pogledaš impl Deref za String
• lahko bi preskočil deserializacijo v JsonValue in deserializiral direkt v struct npr v EpisodesResponse ki ima Vec<Episode> ali pa nek generic container ItemsResponse<T> kjer je T lahko Episodes. (let res = ItemsResponse<Episode>: response.into_json())
  ponekod uwrapaš ko bi imo bilo smiselno handlat error