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

fix: CHARTS-10367 fix code sandbox examples #97

Merged
merged 1 commit into from
Jul 8, 2024

Conversation

mongo-J
Copy link
Collaborator

@mongo-J mongo-J commented Jul 8, 2024

The published SDK release v3.3.0 has a breaking bug which raises an error on load of the SDK. The version is deprecated from npm. Though this shouldn't affect codesandbox examples we have since we have package-lock.json files, it looks like the package-lock.json file is ignored and it will pick up minor patched version automatically.

This PR Updates the sdk versions in the examples to the fixed version to resolve this issue.

Before

https://codesandbox.io/s/github/mongodb-js/charts-embed-sdk/tree/master/examples/charts/unauthenticated
Screenshot 2024-07-08 at 1 07 32 PM

After

https://codesandbox.io/s/github/mongodb-js/charts-embed-sdk/tree/fix/CHARTS-10367-code-sandbox-examples-not-working/examples/charts/unauthenticated
Screenshot 2024-07-08 at 1 07 40 PM

@mongo-J mongo-J added the bug Something isn't working label Jul 8, 2024
@mongo-J mongo-J self-assigned this Jul 8, 2024
Copy link
Collaborator

@kristinamongo kristinamongo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting. I would expect the ^ to exclude deprecated versions but it looks like it does not. Using the ~ would also work for our case but anyways, we probably would like to resume the ^ when the bson issue is fixed.

@mongo-J mongo-J merged commit 66a1997 into master Jul 8, 2024
1 check passed
@mongo-J mongo-J deleted the fix/CHARTS-10367-code-sandbox-examples-not-working branch July 8, 2024 04:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants