Answer:5 # feat: implement tanstack query in todos #772
Conversation
sefatanam
changed the title
sefatanam
changed the title
| onError: (error) => { | ||
| console.error(error); | ||
| }, | ||
| })); |
There was a problem hiding this comment.
You prefer to write it this way, than create a new function for each method ?
There was a problem hiding this comment.
Previously, I attempted a method wherein I declared an empty variable named updateTodos, subsequently modifying it within a switch case, and ultimately setting the query data. However, I found this approach unsatisfactory, thats why I did this modifications. Could you kindly recommend an alternative method for accomplishing this task?
There was a problem hiding this comment.
You prefer to write it this way, than create a new function for each method ?
I understand your idea from your solution. I will need to create a separate mutation function for each action, such as update and delete. This approach should be very straightforward. Thank you
There was a problem hiding this comment.
I need to update my solution, but yes I think I prefer a single function per operation. Easier to maintain
There was a problem hiding this comment.
I've made adjustments to the mutation setup in my todo store, simplifying its usage rather than keeping it completely separate. But I appriciate your solution and will follow in future.
| type: OperationType.DELETE, | ||
| payload: todoId, | ||
| }); | ||
| }); |
apps/angular/crud/src/app/components/todos/todos.component.html
Outdated
Show resolved
Hide resolved
apps/angular/crud/src/app/components/todos/todos.component.html
Outdated
Show resolved
Hide resolved
apps/angular/crud/src/app/components/todos/todos.component.html
Outdated
Show resolved
Hide resolved
|
This pull request is stale because it has been open for 15 days with no activity. |
|
This pull request was closed because it has been inactive for 5 days since being marked as stale. |
This PR includes,