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

Issue fix for #78 #118

Merged
merged 6 commits into from
Mar 6, 2017
Merged

Issue fix for #78 #118

merged 6 commits into from
Mar 6, 2017

Conversation

Lplenka
Copy link
Collaborator

@Lplenka Lplenka commented Mar 4, 2017

Operating Sysytem-Linux Mint
Zulip Desktop Version :1.4.15
Issue : #78

There was no support for localhost zulip servers like this server

I Have fixed this problem
Switching between local and online servers also working fine
Any further suggestions in this issue are welcome

@sinwar
Copy link

sinwar commented Mar 5, 2017

@Lpenka It'll be great if you can create a new branch and make change for pull request.
Also use proper code style and commit style http://zulip.readthedocs.io/en/latest/code-style.html

@Lplenka
Copy link
Collaborator Author

Lplenka commented Mar 5, 2017

@sinwar I will surely take care of these from next time & i have run "npm test run" before PR , I don't think any further issues are left with code

@akashnimare
Copy link
Member

@Lplenka sorry for the delay. This looks fine. Thanks for working on this 🚀

@akashnimare akashnimare merged commit 76381ca into zulip:master Mar 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants