Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

---Final Review Ticket--- #2

Open
boply opened this issue Nov 3, 2022 · 1 comment
Open

---Final Review Ticket--- #2

boply opened this issue Nov 3, 2022 · 1 comment

Comments

@boply
Copy link
Collaborator

boply commented Nov 3, 2022 •
edited by aaravarora01
Loading

Home Page

image

Backend:

Frontend:
@aaravarora01 aaravarora01 mentioned this issue Nov 4, 2022
Open
@jm1021
Copy link

jm1021 commented Nov 4, 2022

The UI and presentation was a bit minimal. It is hard for me to see how you have POST or PUT data going from FE to BE.
I like the use of ArrayList in POJO and Init was well done.
Landing page is nice, but experience beyond that is very simple. Button on cards is confusing.
Lack of comment in code, hard to evaluate key tech without a lot of study of Fetch page
I saw quiz but experience was minimal.
No review tickets shown.
Did not look at commits.

10/12

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jm1021 @boply