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

Justinxue/phs 42 news feed backend edits #49

Closed
wants to merge 73 commits into from

Conversation

jxmoose
Copy link
Collaborator

@jxmoose jxmoose commented Apr 10, 2024

✨ New in this PR ✨

One liner: what did you do? 🛠

small design changes to news feed

Coverage 🙆‍♀️

How can the reviewer test your code? Where is this sprint located in the codebase? 👩‍💻

REQUIRED: Link a short video demo of the feature you have added. 👩‍💻

image

sorry too lazy to record loom lol

Any bugs you encountered or still having trouble with? 🐛

Resources 📔

REQUIREMENTS in this section: Link all relevant online resources and related PRs. Write down any new dependencies or packages and link to their documentation. If you ran into bugs, link any stack overflow or online resources that helped you unblock yourself

❗IF this sprint was marked as FRONTEND on linear, cc Claire Wang

@cluurie

REMEMBER: once PR is made, also request a PR review Sarah, CC'ing is NOT enough!
🏚 cc: @sarahhpeng @andreisito

Copy link

linear bot commented Apr 10, 2024

PHS-42 News Feed

@jxmoose jxmoose closed this Apr 10, 2024
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