search

davidanthoff / Electron.jl

Julia wrapper for Electron
Other
85 stars 19 forks source link

Add sendMessageToJulia to global #80

Closed davidanthoff closed 4 years ago

davidanthoff commented 4 years ago

@arnavgautam and @lrennels, could you try whether this branch works for the thing we just discussed in the meeting? If so, I'll merge and tag a new release.

lrennels commented 4 years ago

Looking at this now, @arnavgautam did you have code to push to the Mimi react explorer branch that makes this check easier? I'm running 

pkg> add Electron#add-sendmsg-to-julia

in my Mimi development environment and then can test but was there a specific spot that it failed before I can target?

lrennels commented 4 years ago

@arnavgautam @davidanthoff ok this works now I think, the compiler now allows the following line. Note that comp_name and item_name are actually variables but since we haven't added the full function in here I made them strings so the compiler didn't get angry.

return function() {
      global['sendMessageToJulia']({cmd: 'display_spec', comp_name: `comp_name`, item_name: `item_name`})
  }