-
Notifications
You must be signed in to change notification settings - Fork 73
Therese Final project #122
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
base: main
Are you sure you want to change the base?
Conversation
…som endpoints, auth and sign in. The last part is not working.
… started created files for data-import
… the data. Import from a CSV-file
…king anymore. Getting data from API but it does not seed the database
…fetching from the API only without combining the data
…CSV and API but no other plants.
… Made a new function, todo-calender. Almost done.
JennieDalgren
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change the title (and maybe favicon)

Some feedback for the user is needed here:

The search result doesnt really look like a finished styling.

Based on what I can see so far you are not approved on this project.
Go through your project again and make the changes needed to fulfill the requirements.
Let me know when this is fixed.
|
ping |

Missing at the moment:
Fully responsive design for devices between at least 320px–1600px
Follows accessibility standards and 100% score in Lighthouse
Follows Clean Code practices
Scrollable items need to be nicer implemented
https://lillebrorgrodafinalproject.netlify.app/
https://garden-backend-r6x2.onrender.com