permafrost06 / DiagMan

Patient/disease case management software
2 stars 1 forks source link

added offline functionality for update & delete #57

Closed protibimbok closed 1 year ago

permafrost06 commented 1 year ago

This pr seems to use localstorage still. I thought we were trying to figure out if an alternative solution (e.g. sqlite in browser) would be better for our case. Please justify.

protibimbok commented 1 year ago

I've used localstorage as it was already implemented, to finish the caching for existing apis. If I switched to SQLite(if possible) right now it'd be a bit messy because this feature is not directly available. I'd had to focus on two different aspects: the SQLite & how it plays with the frontend code. I've somewhat completed the second part so that when we shift to SQLite, it becomes easier (just replacing the localstorage calls with SQLite ones).