API Wrapper Proxy with Closures - Githubissues

CyberCRI / learn-ext

WeLearn Browser Extension

https://welearn.cri-paris.org

MIT License

11 stars 1 forks source link

API Wrapper Proxy with Closures #23

Closed prashnts closed 5 years ago

prashnts commented 5 years ago

I started the work on api-store wrapper to provide a decoupled api interface to the server in previous commit 55038cc8f940cedebfbf5106bbf04d2851f2c0f2

There's a few more changes required to make it fully functional.

Solving the inconsistent states problem

API Store would initialize for each tab specifically.
Network requests are handled exclusively by the wrapper.
Lifecycle methods didAddConcept, didRemoveConcept, didChooseRating, are provided.
The methods make corresponding calls to the root api while managing the internal state themselves.

Closures

The lifecycle methods interface the object with everything inside the closure.
They are supposed to be attached with event handlers wherever required.

prashnts commented 5 years ago

This was implemented in #24