Skip to content
This repository was archived by the owner on Dec 23, 2021. It is now read-only.

server render the metadata#217

Merged
okdistribute merged 2 commits intomasterfrom
server-render-metadata
Sep 8, 2016
Merged

server render the metadata#217
okdistribute merged 2 commits intomasterfrom
server-render-metadata

Conversation

@okdistribute
Copy link
Copy Markdown
Collaborator

@laurengarcia
Copy link
Copy Markdown
Contributor

looks fine, but if user (developer) has the electron-webrtc module installed, Haus is going to use that still, isn't it? lets either nuke it or parameterize it <-- i don't care either way, but the way it is now it is very ambiguous.

@okdistribute
Copy link
Copy Markdown
Collaborator Author

I took out the electron-wrtc part in a separate commit. It needs more thought in haus.js and probably should use hybrid-swarm or a variation that can handle multiple keys, if we are going to use wrtc in the backend.

@okdistribute okdistribute force-pushed the server-render-metadata branch from 173de85 to e7af29d Compare September 6, 2016 08:58
@laurengarcia
Copy link
Copy Markdown
Contributor

ok im cool with this then.

@okdistribute okdistribute merged commit 6f93972 into master Sep 8, 2016
@okdistribute okdistribute deleted the server-render-metadata branch September 8, 2016 09:36
@okdistribute
Copy link
Copy Markdown
Collaborator Author

@laurengarcia okay merged, wondering how it's coming on the archive api with those two methods that will abstract out the stream transport type?

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.

2 participants