Documentation references non existent class SplunkOtelWeb

[SennaSemakula]

Hi team,

I've noticed that there are references to using SplunkOtelWeb class but this doesn't seem to exist in this project. I believe those references should be changed to SplunkRum.

Below and the references I can find:

• https://github.com/signalfx/splunk-otel-js-web/blob/main/examples/installing-npm/README.md?plain=1#L13-L39
• https://github.com/signalfx/splunk-otel-js-web/blob/main/examples/next-ssr-example/instrumentation.client.ts#L21
• https://github.com/signalfx/splunk-otel-js-web/blob/main/examples/next-ssr-example/README.md?plain=1#L133
• https://github.com/signalfx/splunk-otel-js-web/blob/main/examples/installing-npm/src/instrumentation.ts#L20
• https://github.com/signalfx/splunk-otel-js-web/blob/main/packages/web/tests/SplunkOtelWeb.test.ts#L25
• https://github.com/signalfx/splunk-otel-js-web/blob/main/packages/web/tests/api.test.ts#L21C8-L21C21

[amertak]

Hi,

Thanks for pointing that out @SennaSemakula .

It is actually correct as you can name the default import as you want but we will go thru all cases if there is not any mistake.
Official docs is using the SplunkOtelWeb naming as well https://docs.splunk.com/observability/en/gdi/get-data-in/rum/browser/install-rum-browser.html#npm-package

[SennaSemakula]

Hi @amertak.

Thanks for the quick response! Shouldn't it be consistent with the cdn examples e.g. https://docs.splunk.com/observability/en/gdi/get-data-in/rum/browser/install-rum-browser.html#add-the-cdn-script-to-your-application

That uses SplunkRum as well. May make the documentation more consistent and easier for adopters to change if they want to swap the cdn instrumentation for application

[amertak]

Yes, that is definitely something to look into.

I will put this into our backlog and will let you know as we get to it.

Thank you!