JanitorTechnology / c9.ide.janitorconfig

Plugin that reads a janitor.json file and sets up c9 accordingly
0 stars 2 forks source link

Loading Run configurations from janitor.json sometimes fails #4

Closed jankeromnes closed 6 years ago

jankeromnes commented 7 years ago

When loading a Cloud9 IDE window for a Janitor project, very often the console says that the plugin managed to load its janitor.json file successfully, but the Run button has no run configurations.

Refreshing the IDE sometimes fixes this issue. @nt1m were you able to get a review of the Run config loading mechanism from the Cloud9 SDK community?

nt1m commented 7 years ago

There are actually 2 commits trying to fix this: 83be5ebfcf870adf95c1add2029cf91d83770494 and a57b91d03e4a39c23f766def14ae5605f50d9f08

were you able to get a review of the Run config loading mechanism from the Cloud9 SDK community?

According to Harutyun, the way I currently do it (setJson) should work fine: https://groups.google.com/forum/#!topic/cloud9-sdk/Z90iOt-jyo8

but it doesn't unfortunately.

jankeromnes commented 6 years ago

We've been wondering if this issue was gone, but yesterday when I spawned new containers for all 9 Janitor projects, 2 or 3 containers had this bug on first IDE load:

run-button-bug

But since it's become less frequent, I'll rename the issue title to "sometimes".

jankeromnes commented 6 years ago

This seems to work pretty well now. Closing.