search

skyrings / kitoon

Web interface for SkyRing
Apache License 2.0
6 stars 10 forks source link

The curious case of idbWrapper #125

Open debloper opened 8 years ago

debloper commented 8 years ago

I don't see this indexedDB wrapper in action in the application at all. EDIT: iDB has been used once to create the UserRequest store at /app/components/requests/request-tracking-svc.ts#L22 (details undocumented).

Yet, it has been included and bloats up the application (with its library) by significant amount, along with some of the module imports here and there (just imported, and not used).

Are we definitely going to need this for the purpose we're using it for?

debloper commented 8 years ago

Oh, even better... I was replacing it with a leaner workaround. Please do suggest if removal of that part of the code is viable without losing functionality now.

I'll check how the backend implements the tracking and how it depends on the frontend (if at all). But any suggestion would be helpful.