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

feat #559 : Build An Profile Page #601

Merged
merged 1 commit into from
May 26, 2024
Merged

feat #559 : Build An Profile Page #601

merged 1 commit into from
May 26, 2024

Conversation

CoderSwarup
Copy link
Contributor

Description

Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.
building an the profile page for the website where the user can see their personal profile details and other details also

Fixes #(issue_no)

close #559

Screenshots

3

p2

p1

Checklist

  • Tests have been added or updated to cover the changes
  • Documentation has been updated to reflect the changes
  • Code follows the established coding style guidelines
  • All tests are passing

Copy link

netlify bot commented May 25, 2024

👷 Deploy request for counsellor-startup pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 776700f

Copy link

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

Copy link
Contributor

deepsource-io bot commented May 25, 2024

Here's the code health analysis summary for commits 4f0b9b0..776700f. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript❌ Failure
❗ 69 occurences introduced
🎯 12 occurences resolved
View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@thestarsahil
Copy link
Collaborator

Found the usage of undeclared variables

  if (user) {
    console.log("");
  } else if (!user) {
    navigate("/");
  }
});

}, [ ] );

@CoderSwarup
Copy link
Contributor Author

Found the usage of undeclared variables

  if (user) {
    console.log("");
  } else if (!user) {
    navigate("/");
  }
});

}, [ ] );

no need to declare this variable
this is an the passed buy the firebase authentication method as a callback data

as you see same in the dashboard page

image

@thestarsahil
Copy link
Collaborator

Screenshot from 2024-05-25 14-26-44
Screenshot from 2024-05-25 14-26-37

Your Updates effecting to whole dashboard page

  • Effecting to increase in size
  • Responsive side
  • College Card compressed

@CoderSwarup
Copy link
Contributor Author

ok i will fix that

@CoderSwarup
Copy link
Contributor Author

@thestarsahil check it now

Copy link
Collaborator

@thestarsahil thestarsahil left a comment

Choose a reason for hiding this comment

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

For now, it's fine working but we need to improve the code

@thestarsahil thestarsahil added gssoc for GSSOC'24 contributors level3 labels May 26, 2024
@thestarsahil thestarsahil merged commit d52a5a7 into Counselllor:master May 26, 2024
2 of 3 checks passed
@CoderSwarup
Copy link
Contributor Author

For now, it's fine working but we need to improve the code

Sure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gssoc for GSSOC'24 contributors level3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Build a Profile Page
2 participants