Skip to content
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.

Sign up for GitHub

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

Jump to bottom

TaskBuddy - Typescript #48

Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open

TaskBuddy - Typescript #48

wants to merge 17 commits into from

Conversation

Fannyhenriques
Copy link

@Fannyhenriques
started with project to-do -trying out zustand
bae9058
@Fannyhenriques
updated the useTaskStore and added code in the TaskForm and TaskList
7246e88
@Fannyhenriques
wrapped code in useTaskStore in persist to locally store information …
1faf2d5 
…and added a function and a button to remove tasks
@Fannyhenriques
started with some basic styling
982be93
@Fannyhenriques
added styling for list
2ec359e
@Fannyhenriques
added styling
2771bfb
@Fannyhenriques
changed px to rem and changed witdhts
387a67a
@Fannyhenriques
added readme
03d471e
@Fannyhenriques
fixed indentation
74365ad
@Fannyhenriques
fixed indentation
dee0098
@Fannyhenriques
indentation
d06ac75
@Fannyhenriques
initial commit
7ca81f6
@Fannyhenriques
updated usetaskstore with the nessecary typescript
1381e4e
@Fannyhenriques
added HTMLelement on the handlesubmit and onchange in the taskform
54121fb
@Fannyhenriques
added typescript in files tasklist and changed app, main to typescrip…
968a56a 
…t-files and added a declaration.d.ts to import the images
@Fannyhenriques
added a new link in the index.html to connect to the typescript main.tsx
61839c5
@Fannyhenriques
added comments to remember what has been changed
0b91a44
@JennieDalgren JennieDalgren self-assigned this Dec 3, 2024
JennieDalgren
JennieDalgren approved these changes Dec 3, 2024
View reviewed changes
Copy link

@JennieDalgren JennieDalgren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job!

You have converted your code to TypeScript and implemented interfaces in a great way!
All approved

Good work with the commit messages too, easy to follow your process

⭐

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JennieDalgren JennieDalgren JennieDalgren approved these changes

Assignees

@JennieDalgren JennieDalgren

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Fannyhenriques @JennieDalgren