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

Use URL helpers in previews that interact with the server-side #1693

Merged
merged 4 commits into from
Dec 9, 2022

Conversation

camertron
Copy link
Contributor

@camertron camertron commented Dec 8, 2022

Description

Some of our previews don't work in the prod Lookbook because the URLs are hard-coded and only work in dev.

Integration

Does this change require any updates to code in production?

No.

Merge checklist

  • Added/updated tests
  • Added/updated documentation
  • Added/updated previews

@changeset-bot
Copy link

changeset-bot bot commented Dec 8, 2022

🦋 Changeset detected

Latest commit: 7bbb3bf

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@primer/view-components Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@camertron camertron temporarily deployed to review-pr-1693 December 8, 2022 20:03 — with GitHub Actions Inactive
@jonrohan
Copy link
Member

jonrohan commented Dec 8, 2022

Nice! I think we have similar problems with the autocomplete component

@camertron camertron temporarily deployed to github-pages December 8, 2022 20:08 — with GitHub Actions Inactive
@camertron camertron changed the title Use URL helpers for toggle switch previews Use URL helpers in previews that interact with the server-side Dec 8, 2022
@camertron camertron temporarily deployed to github-pages December 8, 2022 22:13 — with GitHub Actions Inactive
@camertron camertron temporarily deployed to github-pages December 8, 2022 22:20 — with GitHub Actions Inactive
@camertron camertron marked this pull request as ready for review December 8, 2022 22:31
@camertron camertron requested a review from a team December 8, 2022 22:31
@jonrohan
Copy link
Member

jonrohan commented Dec 8, 2022

@camertron camertron merged commit 219f5ef into main Dec 9, 2022
@camertron camertron deleted the fix_toggle_switch_previews branch December 9, 2022 21:20
@primer-css primer-css mentioned this pull request Dec 9, 2022
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.

2 participants