search

HackYourFuture-CPH / FP-class11

Class11 Final Project - A collaboration between HYF and Seasony
MIT License
5 stars 8 forks source link

Dashboard Page [Issue 92] #144

Closed cecastosic closed 4 years ago

cecastosic commented 4 years ago

closes #92

mahieakhtar commented 4 years ago

Please update fk_crops_id to fk_crop_id in your knex migration and controller if it's being used because fk_crops_id has been updated to fk_crop_id to make it consistent and to avoid confusion on migrations schema and DB diagram. So please replace fk_crops_id to fk_crop_id if your application is crashing and you want to save time.

zkwsk commented 4 years ago

@cecastosic what is holding this one back? It's been going in and out of review for a long time. Is it possible to get it merged with some missing functionality and creating new PRs for adding anything you might be waiting for? It's a big chunk of the app functionality right here and the sooner we can get it into the app the better :)

cecastosic commented 4 years ago

@cecastosic what is holding this one back? It's been going in and out of review for a long time. Is it possible to get it merged with some missing functionality and creating new PRs for adding anything you might be waiting for? It's a big chunk of the app functionality right here and the sooner we can get it into the app the better :)

I agree 🙂 The only thing that is missing that I am aware of is status box (waiting for endpoints to be merged)...