Review page feature

[nikromen]

This should be more or less the final look of the review page.

Fixes: #4

[xsuchy]

no type?

[nikromen]

The response structure is checked at the return statement with feedback schema - there's just added id for the reference and I didn't want to add another unicorn schema to schema.py since this will disappear with database creation and simply the structure is parsed - just the id is added at the end.

[FrostyX]

@nikromen I see you rebased. Were there any manual merge conflicts or everything went fine?

[nikromen]

@FrostyX there were 2 big merge conflicts with needs of manual resolving - since a lot o fit was on frontend I created another branch for this PR if I screwed since I don't know much about the FE part. But it seems like it's working fine

[TomasTomecek]

I checked it out locally (36c16bd)

and sadly when I click on 'review logs' I get 405:

frontend_1  | shadow-cljs - dependencies updated
frontend_1  | shadow-cljs - HTTP server available at http://localhost:3000
frontend_1  | shadow-cljs - server version: 2.26.7 running at http://localhost:9630
frontend_1  | shadow-cljs - nREPL server started on port 3333
frontend_1  | shadow-cljs - watching build :app
frontend_1  | [:app] Configuring build.
frontend_1  | [:app] Compiling ...
frontend_1  | [:app] Build failure:
frontend_1  | ------ ERROR -------------------------------------------------------------------
frontend_1  |  File: /opt/log-detective-website/frontend/src/app/homepage.cljs:249:20
frontend_1  | --------------------------------------------------------------------------------
frontend_1  |  246 |   [:div {:class "card text-center"}
frontend_1  |  247 |    (render-stats)
frontend_1  |  248 |    (render-navigation)
frontend_1  |  249 |    (render-cards)])
frontend_1  | --------------------------^-----------------------------------------------------
frontend_1  | app/homepage.cljs [line 249, col 20] Unmatched delimiter ).
frontend_1  |
frontend_1  | --------------------------------------------------------------------------------
frontend_1  |  250 |
frontend_1  | --------------------------------------------------------------------------------
frontend_1  |
backend_1   | INFO:     10.89.0.2:58726 - "GET / HTTP/1.1" 200 OK
backend_1   | INFO:     10.89.0.2:58726 - "GET /css/style.css HTTP/1.1" 200 OK
backend_1   | INFO:     10.89.0.2:58728 - "GET /img/fedora.svg HTTP/1.1" 200 OK
backend_1   | INFO:     10.89.0.2:58732 - "GET /static/js/main.js HTTP/1.1" 200 OK
backend_1   | INFO:     10.89.0.2:58732 - "GET /stats HTTP/1.1" 200 OK
backend_1   | INFO:     10.89.0.2:58732 - "GET /img/copr-logo.png HTTP/1.1" 200 OK
backend_1   | INFO:     10.89.0.2:58732 - "GET /img/favicon.ico HTTP/1.1" 200 OK
backend_1   | INFO:     10.89.0.2:58744 - "GET /review/ HTTP/1.1" 307 Temporary Redirect
backend_1   | INFO:     10.89.0.2:58744 - "GET /review HTTP/1.1" 200 OK
backend_1   | INFO:     10.89.0.2:58744 - "GET /frontend/review HTTP/1.1" 405 Method Not Allowed

[TomasTomecek]

We've worked with @nikromen on how to run this locally. Turned out I hate to wipe evrything in frontend/, rebuild images from scratch (--no-cache) and bindmind persisntent volume with :Z.

[TomasTomecek]

Once I submit a review, it's great that I get a message saying

Thank you!
Successfully submitted, thank you for your contribution.

And a link to submit "Another log" which redirects me to /. I know I'm picking nits - would it be better to say "Thank you for your review!", with button "Submit another review" that would immediately take you to another random log.

[TomasTomecek]

I was confused about that directory structure, now I understand :)

[nikromen]

I hope it won't turn into this :D

[TomasTomecek]

Thank you for addressing my comments. Works well for me now!