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

Tech Lead Peer Review #91

Open
ryanhu021 opened this issue Mar 2, 2025 · 1 comment
Open

Tech Lead Peer Review #91

ryanhu021 opened this issue Mar 2, 2025 · 1 comment
Assignees

Comments

@ryanhu021
Copy link
Collaborator

ryanhu021 commented Mar 2, 2025

Description
Discover something new and/or offer meaningful feedback :)

@ktaschek ktaschek self-assigned this Mar 7, 2025
@ktaschek
Copy link

ktaschek commented Mar 7, 2025

Hey Surfrider Tech Leads! Your repo looks really great and well organized. I thought you did a good job writing descriptive issuess, that explained the 'why' and the 'how' for each task well. Additionally, the backlog of prepared issues and the set up project board with different views makes it really clear where you are in development. This is something my team can better incorporate to help devs get a better picture of the full scope of what everyone else is working on. I also really liked the Vercel deployment previews and was looking through some to see progress. The only feedback I can give is that in the form, changing a number value will give a form error because the values are strings and not number.

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

No branches or pull requests

2 participants