-
Notifications
You must be signed in to change notification settings - Fork 2
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
#187419128 users should be able to view and update his/her profile #15
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
rejectValue: ProfileError; | ||
} | ||
>("profile/updateProfile", async (payload: FormData, { rejectWithValue }) => { | ||
try { |
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.
Similar blocks of code found in 2 locations. Consider refactoring.
src/pages/updateProfile.tsx
Outdated
</label> | ||
</div> | ||
|
||
<ProfileTextInput |
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.
Similar blocks of code found in 10 locations. Consider refactoring.
src/pages/updateProfile.tsx
Outdated
name="fullName" | ||
error={errors.fullName?.message} | ||
/> | ||
<ProfileTextInput |
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.
Similar blocks of code found in 10 locations. Consider refactoring.
src/pages/updateProfile.tsx
Outdated
name="gender" | ||
error={errors.gender?.message} | ||
/> | ||
<ProfileTextInput |
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.
Similar blocks of code found in 10 locations. Consider refactoring.
src/pages/updateProfile.tsx
Outdated
name="birthdate" | ||
error={errors.birthdate?.message} | ||
/> | ||
<ProfileTextInput |
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.
Similar blocks of code found in 10 locations. Consider refactoring.
9d33883
to
e0f3dfb
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #15 +/- ##
==========================================
+ Coverage 78.07% 78.86% +0.79%
==========================================
Files 49 56 +7
Lines 789 899 +110
Branches 85 95 +10
==========================================
+ Hits 616 709 +93
- Misses 171 188 +17
Partials 2 2 ☔ View full report in Codecov by Sentry. |
e0f3dfb
to
90c2c64
Compare
90c2c64
to
530509d
Compare
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.
The PR diff size of 16875 lines exceeds the maximum allowed for the inline comments feature.
530509d
to
d7bc920
Compare
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.
The PR diff size of 17150 lines exceeds the maximum allowed for the inline comments feature.
d7bc920
to
9c02cdf
Compare
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.
The PR diff size of 17169 lines exceeds the maximum allowed for the inline comments feature.
9c02cdf
to
e1df745
Compare
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.
The PR diff size of 17109 lines exceeds the maximum allowed for the inline comments feature.
e1df745
to
25aa832
Compare
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.
The PR diff size of 17200 lines exceeds the maximum allowed for the inline comments feature.
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.
The PR diff size of 17268 lines exceeds the maximum allowed for the inline comments feature.
75f5709
to
b11d1f4
Compare
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.
The PR diff size of 17268 lines exceeds the maximum allowed for the inline comments feature.
b11d1f4
to
f4fb310
Compare
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.
The PR diff size of 17268 lines exceeds the maximum allowed for the inline comments feature.
f4fb310
to
b5be3dd
Compare
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.
The PR diff size of 17346 lines exceeds the maximum allowed for the inline comments feature.
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.
LGTM
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.
LGTM
b5be3dd
to
d7682e5
Compare
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.
The PR diff size of 17350 lines exceeds the maximum allowed for the inline comments feature.
d7682e5
to
e76221c
Compare
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.
The PR diff size of 17358 lines exceeds the maximum allowed for the inline comments feature.
e76221c
to
95f2353
Compare
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.
The PR diff size of 17351 lines exceeds the maximum allowed for the inline comments feature.
95f2353
to
b9da233
Compare
-Ensures that user can view his or her profile -Ensure that user can update his or her profile [Deliver #187419128]
b9da233
to
4089e8c
Compare
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.
The PR diff size of 17350 lines exceeds the maximum allowed for the inline comments feature.
Code Climate has analyzed commit 4089e8c and detected 0 issues on this pull request. View more on Code Climate. |
What does this PR do?
User should be able to view or edit his/her profile
Description of Task to be completed?
Have users page to view the profile
Have form to edit profile
How should this be manually tested?
Clone the repo here cd into it .
checkout the branch ft-user-profile-#187419128
Then login to the system. and navigate to http://localhost:5173/profile
What are the relevant pivotal tracker/Trello stories?
#187419128 to see your profile
Screenshots (if appropriate)
User profile view design page
User Profile Figma
Edit Profile desigin page
Update profile figma