-
Notifications
You must be signed in to change notification settings - Fork 1
feat: S3 frontend integration + review flow updates #106
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.
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 6 out of 8 changed files in this pull request and generated no comments.
Files not reviewed (2)
- frontend/app.json: Language not supported
- frontend/package.json: Language not supported
Comments suppressed due to low confidence (1)
frontend/components/MyReview.tsx:143
- The variable 'stepContent' is referenced but not defined in the diff. This will likely cause a runtime error; consider defining 'stepContent' or using the appropriate state variable for the current review step.
const currentStep = stepContent[step - 1];
benjaspet
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We still encounter an issue where, after creating a review, upon clicking it on your user profile it loads infinitely, presumably due to its ID not being populated properly. This needs to be investigated.
The issue was with the frontend call to the review endpoint and the issue now be fixed |
Description
Added to review related components:
Image picking
imagedemo.mp4
Restricting next review screen
Below:
Review submission S3
frontend/utils/s3uploads.tswhich can be reused by other components laterFlow:
Example response from when a review is submitted :
Image uploaded in S3 console:
Image in question:
Adding review image urls to DB
""Intro/outro screens
Intro screen
Outro screen
Checklist
maininto my branch.