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

adjust demo package to fix CodeSandbox setup #220

Merged
merged 1 commit into from
Oct 4, 2023

Conversation

chrisdeely
Copy link
Contributor

@chrisdeely chrisdeely commented Oct 4, 2023

fixes #219
These changes repaired the CodeSandbox referenced within the documentation. I'll admit that I don't know the history/rationale behind directly referencing the GitHub repo as the dependency location, so this may not be an appropriate fix.

Example sandbox

@chrissrogers
Copy link
Member

chrissrogers commented Oct 4, 2023

Thank you for this contrib! I'll merge it once tests pass. Using the github reference was meant to keep the demo as up to date with the main branch as possible, but I think that this approach is much safer and matches production deployments. And clearly the existing github reference doesn't work for codesandbox any more, so this is much better.

@douglasmiller douglasmiller merged commit 8513b5f into recurly:main Oct 4, 2023
3 checks passed
@chrisdeely chrisdeely deleted the patch-1 branch October 4, 2023 23:55
recurly-integrations pushed a commit that referenced this pull request Oct 12, 2023
[Full Changelog](v2.0.0...v2.0.1)

**Merged Pull Requests**

- adjust demo package to fix CodeSandbox setup [#220](#220) ([chrisdeely](https://github.com/chrisdeely))
- Adds hostname configuration [#217](#217) ([chrissrogers](https://github.com/chrissrogers))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ineractive Demo page not working properly
3 participants