Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Firefox #258

Closed
wants to merge 5 commits into from
Closed

WIP: Firefox #258

wants to merge 5 commits into from

Conversation

bdougie
Copy link
Member

@bdougie bdougie commented Aug 15, 2023

This is a WIP from @nickytonline's forked branch. I ran this on stream to identify the error message my self and also learn how to debug this locally in firefox.

Opening this PR as spike.

Screen Shot 2023-08-15 at 2 38 45 PM

@bdougie
Copy link
Member Author

bdougie commented Aug 15, 2023

Current Error: chrome.storage

#237 (comment)

I think I might have narrowed to the chrome.storage in the settings.tsx. There is a storage.local that might be needed instead, for firefox. https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/storage/local

https://github.com/open-sauced/ai/blob/beta/src/popup/pages/settings.tsx#L22

Suggested solution

But TBH, we probably should use a state management library in React instead

@bdougie
Copy link
Member Author

bdougie commented Sep 18, 2023

@nickytonline thanks for picking this up.

@bdougie bdougie closed this Aug 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants