-
Notifications
You must be signed in to change notification settings - Fork 639
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
Local WOPI for debug builds #3226
Conversation
fd3dc15
to
7c45373
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Rash419 Thanks so much, this is great! To be able to integrate it, I'd like to ask you for few improvements from the code reading perspective - but nothing substantial :-)
One more thing that I didn't explicitly point to in the review; the C++ bits use a different formatting than the JS - the {'s and }'s are on new lines, please have a look at the other parts of the file reformat the code accordingly too.
1c06a96
to
7becb4a
Compare
…cal storage" (in debug builds) Signed-off-by: Rash419 <rashesh.padia@collabora.com> Change-Id: I9e21df4c5b611ca9989915180310f0991ed28725
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Rash419: This is awesome, thanks a lot, makes the WOPI testing much easier! :-)
Signed-off-by: Rash419 rashesh.padia@collabora.com
Change-Id: Ia5cfcc4c345ad062ac3e93b0e4f9bc6321781b63
*github issue: https://github.com/CollaboraOnline/online/issues/110