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

Update desktop build instructions fixes #2792 #2793

Merged
merged 1 commit into from Dec 22, 2016

Conversation

@joelostblom
Copy link
Contributor

joelostblom commented Dec 22, 2016

This is based on what I had to do to get the desktop app running, as per the advice in #2792. I am not familiar with npm, so please change this pull request if there is a more appropriate way of achieving the same result.

This is based on what I had to do to get the desktop app running, as per the advice in #2792. I am not familiar with `npm`, so please change this pull request if there is a more appropriate way of achieving the same result.
@ara4n

This comment has been minimized.

Copy link
Member

ara4n commented Dec 22, 2016

oops - thanks for this!!

@ara4n

This comment has been minimized.

Copy link
Member

ara4n commented Dec 22, 2016

i'll merge to master and cherrypick to develop.

@ara4n ara4n merged commit e8eadc5 into vector-im:master Dec 22, 2016
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@joelostblom joelostblom deleted the joelostblom:patch-1 branch Dec 22, 2016
andreygursky added a commit to andreygursky/riot-web that referenced this pull request Feb 16, 2017
andreygursky added a commit to andreygursky/riot-web that referenced this pull request Feb 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.