Conversation
pheobeayo
requested review from
Teddy1792,
joelamouche and
oscarwroche
as code owners
joelamouche
left a comment
joelamouche
left a comment
There was a problem hiding this comment.
Overall looks good but I wasn't able to test, see comment
frontend/src/lib/types/projects.d.ts
Outdated
| id: string; | ||
| name: string; | ||
| description: string; | ||
| owner_address: string; |
There was a problem hiding this comment.
If you look at backend/src/application/dtos/project_dtos.rs you will see that the api doesn't return owner_address.
This is why when testing locally I get
Also if I look at backend/migrations/004_create_projects_table.sql I can see this field is not in the db either.
Please add owner_address in the backend to fix this pb.
Also, in the future, pls run and test locally your app before posting a PR.
There was a problem hiding this comment.
Okay, I will work on it
joelamouche
left a comment
joelamouche
left a comment
There was a problem hiding this comment.
I can't stress this enough: you need to test features locally before submitting a PR for review
|
|
||
| export type CreateProjectData = { | ||
| name: string; | ||
| description: string; |
There was a problem hiding this comment.
status is missing here but required in backend/src/application/dtos/project_dtos.rs whichy is why I'm getting this
2 participants
fixes #166