Open issue from pop-up goes to openworm

[dosumis]

=> issue on openworm/org.geppetto

Should go here - incorporating log.

[ddelpiano]

Open issue voice on the bottom point to openworm at the moment, to be changed to point to the VFB.

[ddelpiano]

@jrmartin the issue sits in geppetto-client/js/components/controls/modals/ErrorModal.js, in order to fix this we need to be able to inject 2 props:

• a bool to get the button on or off depending on what the user prefers.
• the url we want to use to open the issue (please redirect to https://github.com/VirtualFlyBrain/geppetto-vfb/issues).

Since we need to do this for the "Open issue" button I would the same also for the tweet button "Shame on you" so the user can decide to enable/disable both the button and configure them.

A way I thought to do this could be to re-use the GeppettoConfiguration.json (or istantiate another configuration) where we have something similar:

{
....
  errorDialog: {
    twitterButton: {
      enabled: true
      url: 'https://twitter.com/openworm'
      message: 'I broke geppetto, ops!'
    }
  }
}

So the same structure can be used also for the other modals in a second moment. The error modal will try to get this info from the GeppettoConfiguration (or similar) but if this is not available it has a default configuration in the component itself to use if this is not provided (the geppetto one). I would go for something similar like this if @tarelli it's ok.

[tarelli]

Sounds reasonable, I wouldn't add a new configuration though. And currently everything is flat within the GeppettoConfiguration but this isn't necessarily a problem. Create a card if it's not there already in geppett-client for this addition. Once you guys have the final proposed change to the file put it there for review, we can also discuss at the next Geppetto call.

[jrmartin]

First proposed changes to make this happen are done, waiting on review of PR https://github.com/openworm/geppetto-client/pull/111.