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

browser-add-file #364

Merged

Conversation

nycoliver
Copy link

Example showing broken file add in browser. See #344 (comment) and #335 for context.

@nycoliver
Copy link
Author

Tests failing because FileReader is undefined in node environment. Not sure how to fix...

@nginnever
Copy link
Contributor

nginnever commented Sep 4, 2016

Thanks for posting this @nycoliver

Tests failing because FileReader is undefined in node environment. Not sure how to fix...

That's just a linting error, so really just lint doesn't know about FileReader global since node never runs that example code. You could just instantiate a FileReader object at the scope you need with something like let FileReader

@daviddias
Copy link
Contributor

@nycoliver please add /* globals FileReader */at the top and squash the commits for the merge :)

@daviddias
Copy link
Contributor

@nycoliver looks all good now. Can you coalesce the commits?

git rebase -i HEAD~6
# squash them all
# reword the commit message to `feat(examples): add browser-add example` 

@nycoliver
Copy link
Author

First pull request, almost got it...

@daviddias
Copy link
Contributor

Here is a good tutorial http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html let me know if you run into trouble, I'm available on IRC as daviddias on the #IPFS channel in freenode

@daviddias
Copy link
Contributor

Congrats on your first pull request 🎉🎉🎉🎉👏👏👏👏👏👏👏👏👏

@daviddias daviddias merged commit 740aaad into ipfs-inactive:fixes-browser-add Sep 8, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants