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

HER-45 Speaker Index Page #47

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

HER-45 Speaker Index Page #47

wants to merge 66 commits into from

Conversation

mhope21
Copy link
Collaborator

@mhope21 mhope21 commented Jan 8, 2025

HER-45: Create Speaker Index Page [FE]

Issue

https://raquelanaroman.atlassian.net/browse/HER-45

Description

As a teacher user, I want to browse all speakers to find someone suitable for my event.

Deliverables:

Display speaker cards with name, bio, image, and a link to profile.

Basic layout and styling for speaker cards.

Mock data for testing.

Acceptance Criteria

The Speakers page displays each speaker user with the following information: name, bio, image, and a “View Profile” button.

Clicking “View Profile” navigates to speaker user’s profile page.

https://raquelanaroman.atlassian.net/browse/HER-46
https://raquelanaroman.atlassian.net/browse/HER-47

Changes

  1. Add speaker index page with cards for each speaker displaying image, name, bio, and profile button.
  2. Change profile action to find profile by id of the user, not current_user.
  3. Add seed data and images to test speaker index page.
  4. Modify user serializer and user profile serializer to display attached images.
  5. Modify css styles.
  6. Add a checkbox for "I am a speaker" to the registration page.

Review Checklist

  • I have documented my code with code comments.

Copy link
Collaborator

@trca831 trca831 left a comment

Choose a reason for hiding this comment

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

LGTM

Marcia added 27 commits January 8, 2025 11:12
@mhope21
Copy link
Collaborator Author

mhope21 commented Jan 8, 2025

Sorry about all these commits. Most of them are from doing rebase every time main changed.

@raquii raquii changed the base branch from main to HER-14-Create-name-method-for-User January 9, 2025 02:45
@raquii raquii changed the base branch from HER-14-Create-name-method-for-User to main January 9, 2025 02:45
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