Ace edit json dialog and redis

[jmthomas]

closes #583

[codecov]

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.07 🎉

Comparison is base (d4a8e96) 74.41% compared to head (5823710) 74.48%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #608      +/-   ##
==========================================
+ Coverage   74.41%   74.48%   +0.07%     
==========================================
  Files         458      457       -1     
  Lines       28262    28270       +8     
  Branches      589      588       -1     
==========================================
+ Hits        21031    21057      +26     
+ Misses       7137     7120      -17     
+ Partials       94       93       -1     

Flag	Coverage Δ
frontend	75.16% <100.00%> (+0.41%)	⬆️
ruby-api	51.02% <ø> (ø)
ruby-backend	78.67% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...riptrunner/src/tools/ScriptRunner/ScriptRunner.vue	69.72% <100.00%> (ø)

... and 8 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[ryanmelt]

Using id is going to cause us problems eventually if we do the split screen. Is there any way to not use id, or give it a unique id?

[jmthomas]

Yeah I think you can select the element yourself and give it to the ace constructor. I'll see if I can make that work and update EditScreenDialog and ScriptRunner as well.