Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clarify current state of browser support in the README #4035

Closed
pichlermarc opened this issue Aug 2, 2023 · 4 comments · Fixed by #4037
Closed

Clarify current state of browser support in the README #4035

pichlermarc opened this issue Aug 2, 2023 · 4 comments · Fixed by #4037
Labels
document Documentation-related feature-request

Comments

@pichlermarc
Copy link
Member

Currently, browser support of the packages in this repo is not considered production-ready

We should:

@chalin
Copy link
Contributor

chalin commented Aug 2, 2023

The status given in Status and Releases of the JS page of OTel.io is:

image

Would it make sense to change it to the following?

Some heading Traces Metrics Logs
Node.js Stable Stable Development
Browser Experimental Experimental Development

/cc @svrnm @cartermp

@svrnm
Copy link
Member

svrnm commented Aug 4, 2023

open-telemetry/opentelemetry.io#3125 got merged, you can take the wording used on the website for the README if you like (or come up with better words and let us know and we use them as well)

@pichlermarc
Copy link
Member Author

The status given in Status and Releases of the JS page of OTel.io is:

image

Would it make sense to change it to the following?
Some heading Traces Metrics Logs
Node.js Stable Stable Development
Browser Experimental Experimental Development

Hmmm 🤔 From my understanding, this matrix always represented the stability of the interface, not the runtime support. I wouldn't rule out that users interpret this in another way, though. I'll ask about this in the SIG Meeting next week to see what others think. 🤔

cc @open-telemetry/javascript-approvers

@pichlermarc
Copy link
Member Author

open-telemetry/opentelemetry.io#3125 got merged, you can take the wording used on the website for the README if you like (or come up with better words and let us know and we use them as well)

thanks, opened #4037. I think this fits in nicely without modifications. 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
document Documentation-related feature-request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants