issues
search
NickPerlich
/
Bettermo
1
stars
1
forks
source link
Schema/API Design comments Kaveh Ghalambor
#7
Open
kghalamb
opened
11 months ago
kghalamb
commented
11 months ago
Going to your home page reveals a row when it probably shouldn't
There seems to be a lot of tables that are not listed in your ER diagram
The docs page says Central Coast Cauldrons
put descriptions for all of your endpoints to make it more readable
add more endpoints for sanity checking, getting users, getting groups etc
There is no way to edit rows, which would definitely be helpful
Make an endpoint to delete a row to give you more control over your data
you could add more error handling for api endpoints, more than just a boolean value
make the description for api payloads more descriptive
make some default values for inputs, can keep bad input from being destructive
Based on your ER Diagram, you should try to add more columns to handle the amount of data that your app needs to handle
try to be more consistent with how you name variables, some are named with camelBack, while others use underscorts