-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
ux friendly use of remixd inside electron #2354
Conversation
d0d024c
to
f84f872
Compare
console.log(error) | ||
modalDialogCustom.alert( | ||
'Cannot connect to the remixd daemon.' + | ||
'Please make sure you have the remixd running in the background.' |
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.
Is it run by default ? Maybe could be good to have a link to documentation.
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.
yep, there's already a link in the content of the modal
src/app/files/remixd-handle.js
Outdated
@@ -85,6 +90,10 @@ export class RemixdHandle extends Plugin { | |||
} | |||
} | |||
) | |||
} else { | |||
this.locahostProvider.init((error) => { |
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.
Could be on one line : this.locahostProvider.init(error => connection(error))
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.
updated
@@ -19,11 +19,14 @@ export class RemixAppManager extends PluginEngine { | |||
constructor (plugins) { | |||
super(plugins, settings) | |||
this.event = new EventEmitter() | |||
this.donotAutoReload = ['remixd'] // that would be a bad practice to force loading some plugins at page load. |
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.
I'm not sure to understand what it does.
f84f872
to
a4e24d2
Compare
merge after #2348