search

alltrust / Reminder-App

Set reminders based importance. Search & filter through passed reminders
0 stars 0 forks source link

Losing State #1

Open sebastianserrano opened 2 years ago

sebastianserrano commented 2 years ago

How come you're not resetting the old state? Wouldn't you be losing info?

https://github.com/alltrust/Reminder-App/blob/628a2d8c14e37277dc36ce95243ad413ef014cd7/src/store/ui-actions.js#L31

Like you did here

https://github.com/alltrust/Reminder-App/blob/628a2d8c14e37277dc36ce95243ad413ef014cd7/src/store/ui-actions.js#L15

alltrust commented 2 years ago

I have yet to include this feature in the app- will do soon

sebastianserrano commented 2 years ago

I have yet to include this feature in the app- will do soon

Sounds good. Make an issue describing what you will be doing first and then create a PR linking back to the issue