Using React for frontend and Django Rest Framework for backend

Community feedback

• 

  ChrishanZ•0

  @ChrishanZPosted almost 3 years ago

  Very good, can you try to fill the "all" filter ? or fill the selected one , really nice job !

  1

  ---

  

  P

  Radomir Mijovic•150

  @radomir-mijovicPosted almost 3 years ago

  @ChrishanZ Hi, thank you for you feedback. I am not sure if I understand your question. Can you please explain me a little bit better your question? Thanks

  0

  

  ChrishanZ•0

  @ChrishanZPosted almost 3 years ago

  @radomir-mijovic I mean, on the left side, there is some filters (all, UX, UI ...), i was telling if you could fill the selected one, it would be perfect, cheers. :)

  0

  

  P

  Radomir Mijovic•150

  @radomir-mijovicPosted almost 3 years ago

  @ChrishanZ I see, they are working on desktop but not on mobile for some reason. I will check. Thanks for that.

  0
• 

  JK•345

  @JakubKepakPosted almost 3 years ago

  Very nice job Radomir! I've noticed a small thing which might take your solution even higher, in my opinion :).

  Sometimes you use under score in variables names. For instance, up_vote in In ButtonUpVote.js. I would stick with camelCase only, it's more consistent.

  Cheers!

  0

  ---

  

  P

  Radomir Mijovic•150

  @radomir-mijovicPosted almost 3 years ago

  @JakubKepak thanks for reply and I'm glad that you liked my solution, this up_vote is coming from backend and convention to write variables names in Django(Python) is usually like that, as far as I know :) . I didn't want to create separate variables when I can use backend variables for updating my state. In React I always use camelCase. I really appreciate your comment. Thanks

  0