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.

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

Merged
merged 1 commit into from
Jul 6, 2024

Conversation

niyobertin
Copy link
Collaborator

@niyobertin niyobertin commented Jun 27, 2024

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

  • run npm install to install
  • run npm dev to start server

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
Screenshot 2024-07-03 at 15 25 43

User Profile Figma
Screenshot 2024-07-03 at 15 30 53

Edit Profile desigin page
Screenshot 2024-07-03 at 15 26 12

Update profile figma
Screenshot 2024-07-03 at 15 30 33

Copy link

vercel bot commented Jun 27, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
eagles-ec-fe 🛑 Canceled (Inspect) Jul 5, 2024 5:54pm

rejectValue: ProfileError;
}
>("profile/updateProfile", async (payload: FormData, { rejectWithValue }) => {
try {
Copy link

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.

</label>
</div>

<ProfileTextInput
Copy link

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.

name="fullName"
error={errors.fullName?.message}
/>
<ProfileTextInput
Copy link

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.

name="gender"
error={errors.gender?.message}
/>
<ProfileTextInput
Copy link

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.

name="birthdate"
error={errors.birthdate?.message}
/>
<ProfileTextInput
Copy link

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.

@codecov-commenter
Copy link

codecov-commenter commented Jun 27, 2024

Codecov Report

Attention: Patch coverage is 84.11215% with 17 lines in your changes missing coverage. Please review.

Project coverage is 78.86%. Comparing base (9652708) to head (4089e8c).

Files Patch % Lines
src/redux/reducers/profileSlice.ts 77.77% 6 Missing ⚠️
src/redux/reducers/updateProfileSlice.ts 75.00% 6 Missing ⚠️
src/pages/userProfile.tsx 84.84% 4 Missing and 1 partial ⚠️
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.
📢 Have feedback on the report? Share it here.

Copy link

@codeclimate codeclimate bot left a 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.

@niyobertin niyobertin force-pushed the ft-user-profile-#187419128 branch from 530509d to d7bc920 Compare July 1, 2024 13:07
Copy link

@codeclimate codeclimate bot left a 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.

Copy link

@codeclimate codeclimate bot left a 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.

Copy link

@codeclimate codeclimate bot left a 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.

Copy link

@codeclimate codeclimate bot left a 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.

Copy link

@codeclimate codeclimate bot left a 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.

Copy link

@codeclimate codeclimate bot left a 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.

@niyobertin niyobertin force-pushed the ft-user-profile-#187419128 branch from b11d1f4 to f4fb310 Compare July 3, 2024 19:12
Copy link

@codeclimate codeclimate bot left a 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.

Copy link

@codeclimate codeclimate bot left a 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.

Copy link
Collaborator

@soleil00 soleil00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Collaborator

@yvanddniyo yvanddniyo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@niyobertin niyobertin force-pushed the ft-user-profile-#187419128 branch from b5be3dd to d7682e5 Compare July 4, 2024 16:15
Copy link

@codeclimate codeclimate bot left a 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.

Copy link

@codeclimate codeclimate bot left a 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.

@niyobertin niyobertin force-pushed the ft-user-profile-#187419128 branch from e76221c to 95f2353 Compare July 5, 2024 13:25
Copy link

@codeclimate codeclimate bot left a 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.

-Ensures that user can view his or her profile
-Ensure that user can update his or her profile

[Deliver #187419128]
Copy link

@codeclimate codeclimate bot left a 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.

Copy link

codeclimate bot commented Jul 5, 2024

Code Climate has analyzed commit 4089e8c and detected 0 issues on this pull request.

View more on Code Climate.

@teerenzo teerenzo merged commit 548eca2 into dev Jul 6, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants