addyosmani / addyosmani/clipai

Remove background.js?

Aperta
#14 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
JavaScript
Stelle
38
Fork
7
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

At this point, it looks like `background.js` is not being used for anything. I believe it's safe to remove in the current incarnation of the extension.

Alternatively, we could keep `background.js` and use it as a data provider for clips. That would involve updating `sidebar.js` to send messages to `background.js` for this purpose instead of reading/saving clips itself.

Let me know which approach you'd like me to take.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.