Skip to content
This repository has been archived by the owner on Mar 17, 2024. It is now read-only.

Browser support? #3

Closed
ryndshn opened this issue Mar 6, 2016 · 1 comment
Closed

Browser support? #3

ryndshn opened this issue Mar 6, 2016 · 1 comment

Comments

@ryndshn
Copy link

ryndshn commented Mar 6, 2016

According to the reddit API wiki, this wrapper has browser support. But I don't see any evidence of that here... Here is the page I'm referring to: https://github.com/reddit/reddit/wiki/API-Wrappers. Does snoowrap have browser support?

@not-an-aardvark
Copy link
Owner

not-an-aardvark commented Mar 6, 2016

I believe snoowrap should work with browserify without needing to make any modifications to the code. For example, there aren't any methods that demand access to the filesystem. The methods that can access the filesystem (such as upload_header_image) all have alternative options so that they can work in a browser environment.

That said, I haven't yet gotten around to figuring out unit tests for browser support, so it's possible that something related to browser support has broken since I last tested it manually.

[edited a few weeks later]: There are now browserified files here and here (minified). You can test snoowrap in your browser by using the snoowrap global exposed in the dev console while on the documentation site.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants