bracket icon indicating copy to clipboard operation
bracket copied to clipboard

Add member names to teams table

Open jmillxyz opened this issue 7 months ago • 1 comments

Hi there, cool project. I appreciated seeing a working example of FastAPI + NextJS together.

I noticed the teams table didn't show each team member in the members column, so here's a small PR to do so. Feel free to suggest edits!

Screenshot: Screenshot 2025-05-05 at 12 24 51 PM

jmillxyz avatar May 05 '25 17:05 jmillxyz

@jmillxyz is attempting to deploy a commit to the evroon's projects Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar May 05 '25 17:05 vercel[bot]

Hey there! I am new to APIs, why are you using FastAPI ad nextJS together and what does it actually accomplish in this project? Thank you for your time in advance!

Actay893 avatar May 08 '25 03:05 Actay893

Hey there! I am new to APIs, why are you using FastAPI ad nextJS together and what does it actually accomplish in this project? Thank you for your time in advance!

It's best to start a discussion with the Q&A topic for that. I will answer it there, for this PR it's a bit off-topic :)

evroon avatar May 08 '25 08:05 evroon

Thanks for the fix, I think it looks good! There was a reason I disabled the player names in the past, not sure why anymore, probably because I temporarily excluded them from the backend response to make the endpoint simpler/faster, but that's not necessary anymore.

evroon avatar May 08 '25 08:05 evroon