[Console] Fix example in help flyout

[alisonelizabeth]

Fixes #69253

The example .txt file was imported incorrectly.

Release note

This fixes an issue in Dev Tools Console where the example shown in the "Help" panel was formatted incorrectly.

[elasticmachine]

Pinging @elastic/es-ui (Team:Elasticsearch UI)

[cjcenizal]

Code LGTM, didn't test locally. WDYT of hardcoding the string in JS as a string literal to preserve whitespace? I think this would simplify things by removing the dependency on a webpack loader and removing a .txt file, which we don't use anywhere else AFAIK so it's a bit of an oddity. It would also prevent this bug from happening again.

[jloleysens]

@alisonelizabeth Thanks for fixing this! I wonder when this happened and if it has to do with the use of import. In that case this may have been introduced after the NP migration.

I see that pre NP migration it was imported without !!raw-loader! (https://github.com/elastic/kibana/blob/6.5/src/core_plugins/console/public/src/directives/sense_help_example.js). Not that it matters too much :)

I tend to agree with CJ that we can probably just keep this in a const helpText =. But will leave it up to you!

[alisonelizabeth]

@cjcenizal great suggestion! I updated the PR with this change.

[alisonelizabeth]

@elasticmachine merge upstream

[alisonelizabeth]

@elasticmachine merge upstream

[alisonelizabeth]

@elasticmachine merge upstream

[kibanamachine]

💚 Build Succeeded

• continuous-integration/kibana-ci/pull-request
• Commit: 03f3bd3

Build metrics

✅ unchanged

History

• 💔 Build #60217 failed a3951bb
• 💔 Build #60173 failed 3e36b9b
• 💔 Build #60132 failed 7ce5931
• 💛 Build #60040 was flaky a3f504b

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream