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

Profile and Templates setup #61

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

jhansilakshmi27
Copy link

PR Description

Profile Page Enhancements

  • Personal Information Display: Shows user details, including name, branch, college, and an "About" section.
  • Connect Feature: Allows users to connect with others on the platform.
  • Course List: Displays a list of courses associated with the user.
  • Discussion Page Link: Provides easy navigation to the discussion page.
  • Settings Section: Added a section for managing user preferences.
  • Edit Option: Users can edit and update their profile details.
  • Updated Navbar: Added a profile page link for quick access.
  • UI Enhancements: Improved user experience with smooth hover effects and transitions.

image

Template Page Implementation

  • Pop-ups after Login: Added user-friendly pop-ups post-login for template creation.
  • Template Card Feature: Enabled users to create and customize their own template cards.

image
image

@gunjjoshi
Copy link
Collaborator

@jhansilakshmi27 A few comments regarding this PR:

  • The profile page is still blank, just as it was in the previous PRs.
  • The template feature seems to be misunderstood. What we want is:
    • Users should be asked to select their college, out of the available list of colleges.
    • Based on the college they select, they should only see those Notes, PYQs, ratings, etc., which belong to that particular college.
    • If their college doesn't exist in our list yet, an admin from their college should register themselves with us. Once their college is registered, it would be visible in the dropdown.
    • We'll need an admin authorization as well.
  • This PR has some merge conflicts, please have a look over them.

Overall, it seems that still there is some work to be done on this PR. @jhansilakshmi27 You might want to rethink the whole template architecture. Thanks for working on this!

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