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

internal(vx-demo): sandbox cleanup #727

Merged
merged 6 commits into from
May 23, 2020
Merged

Conversation

williaster
Copy link
Collaborator

@williaster williaster commented May 22, 2020

🏠 Internal

This PR cleans up some remaining items for the demo codesandbox refactor and should close #624 🎉 It specifically

  • moves src/v2-docs/examples/* => src/sandboxes/*
  • adds "private": true" to each sandbox package.json since they shouldn't be published
  • removes some unused .js files in src/components/codeblocks,charts,util

Testing

  • CI
  • functional
    • verified the Gallery still works (which should test each example)
    • verified the new CodeSandboxLink link works (example, update to master branch upon merge)

@hshoff @kristw

@williaster williaster added this to the v1 milestone May 22, 2020
@williaster williaster merged commit 79a9c92 into master May 23, 2020
@williaster williaster deleted the chris--sandbox-cleanup branch May 23, 2020 04:45
@hshoff hshoff modified the milestones: v1, v0.0.197 Jun 4, 2020
@ryee-dev
Copy link

ryee-dev commented Oct 9, 2020

Just tried the codesandbox link for Geo Mercator on a few different browsers and I'm not able to get it to load for some reason..

Screen Shot 2020-10-09 at 12 27 18 AM

Just wanted to check if this issue is isolated to just me.

@williaster
Copy link
Collaborator Author

@ryee-dev are you using the link from https://vx-demo.now.sh/ ?

Sorry, we need a deprecation notice for that. The demo has now moved to airbnb.io/visx, and the sandbox links have changed https://airbnb.io/visx/geo-mercator now links to https://codesandbox.io/s/github/airbnb/visx/tree/master/packages/visx-demo/src/sandboxes/visx-geo-mercator

Hope that helps.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[docs] link out to codesandbox from examples
3 participants