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

Ability to stop auto update #2617

Merged
merged 1 commit into from
Nov 21, 2019
Merged

Ability to stop auto update #2617

merged 1 commit into from
Nov 21, 2019

Conversation

hsiehjack
Copy link
Contributor

@hsiehjack hsiehjack commented Nov 17, 2018

Fix #2140

image

@daiyam
Copy link
Contributor

daiyam commented Nov 18, 2018

@hsiehjack please use the existing config (browser/main/lib/ConfigManager.js)

@ZeroX-DG ZeroX-DG added the awaiting review ❇️ Pull request is awaiting a review. label Nov 18, 2018
@daiyam
Copy link
Contributor

daiyam commented Dec 16, 2018

@hsiehjack Are you planning the make the change? It would be good to have your PR.

@hsiehjack
Copy link
Contributor Author

@daiyam I made the changes already. Looks like there was a build fail. I will have to take a look at what happen.

@RonWalker22
Copy link
Contributor

Any updates on this?

@hsiehjack hsiehjack force-pushed the issue-2140 branch 2 times, most recently from 6f3417f to bdd03c4 Compare August 6, 2019 07:59
@hsiehjack
Copy link
Contributor Author

Sorry for the long wait. This is ready for review.

@Flexo013
Copy link
Contributor

Flexo013 commented Nov 6, 2019

@hsiehjack It looks like there are more conflicts. Could you resolve these, then we'll get this approved asap?

@hsiehjack
Copy link
Contributor Author

@Flexo013 conflict fixed.

@Rokt33r
Copy link
Member

Rokt33r commented Nov 6, 2019

I'll review this tomorrow!

@Rokt33r
Copy link
Member

Rokt33r commented Nov 18, 2019

Sorry I've been insanely busy recently... I'll try to test it tonight or tomorrow. No more delaying I promise..! 🙏

@Rokt33r
Copy link
Member

Rokt33r commented Nov 21, 2019

Confirmed that it's working well on macOS. I'm testing on WIndows now.

@Rokt33r Rokt33r removed the awaiting review ❇️ Pull request is awaiting a review. label Nov 21, 2019
@Rokt33r Rokt33r added this to the v0.14.0 milestone Nov 21, 2019
@Rokt33r Rokt33r merged commit d78f6b7 into BoostIO:master Nov 21, 2019
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.

Update should be optional
6 participants