HackYourFuture-CPH / FP-class11

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

chart-detail-smart-page #182

Closed mamathamereddy closed 4 years ago

mamathamereddy commented 4 years ago

resolves #125
---> progress bar data ---> sidebar - dashboard button click would open list items like temperature, ph ... ---> detail chart - data is displayed based on the days of batch. ---> dashboard list items - would open the respective detail chart like temperature, ph ... ---> filters data based on buttons last5days, lastWeeks and custom range. Hierarchy :
Smart Component:
containers -chart-detail-page

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.

sowmya1408 commented 4 years ago

There are a lot of problems in this PR.

Its not the problem of PR, but there was change from fk_crops_id to fk_crop_id in crop_stages table in database, its effecting the endpoints we are using. It happened also with @Kristina who reviewed our PR, she did a db clean and then db setup and it worked :)

cecastosic commented 4 years ago

Title is out of viewport, and it is half visible and it is too big.

Screenshot 2020-04-26 at 13 50 48

I think you fixed buttons background, after I did screenshot.