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

View profile - add 'View Profile' button for mentors to access their own profile #848

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

code-par-learn
Copy link
Contributor

Related Issue

Created a 'View Profile' button for mentors to access their own profile

Description of Changes

  • updated the /pages/dashboard/mentor.page.js , to include a new tab that says view-profile
  • updated the /components/mentorDashboard/home.js , to include the button
  • created a new file /components/mentorDashboard/ViewProfile.js , to render the profile view

Checklist:

  • My code adheres to the established style guidelines of this project.
  • I have conducted a self-review of my code.
  • I have included comments in areas that may be difficult to understand.
  • I have made corresponding updates to the project documentation.
  • My changes have not introduced any new warnings.

Screenshots

Original Updated
Original screenshot Updated screenshot

Please provide any necessary screenshots to illustrate the changes made.

Copy link

vercel bot commented Aug 27, 2024

@code-par-learn is attempting to deploy a commit to the anmode's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

Thank you, @code-par-learn, for creating this pull request and contributing to GitHub-ReadMe!
Our review team will thoroughly review the pull request and will reach out to you soon!
Please make sure you have marked all the completed tasks as done.
We appreciate your patience and contribution!

Copy link
Owner

@anmode anmode left a comment

Choose a reason for hiding this comment

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

Looks good to me, We'll merge once that btn issue will be solved in this PR

@code-par-learn
Copy link
Contributor Author

Looks good to me, We'll merge once that btn issue will be solved in this PR

sure , I'll try to resolve it soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants