Skip to content
New issue

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

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

Already on GitHub? # to your account

[PLAY-1871] Playbook Website Bug - React Several Hot Reloads #4326

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

cmhorsey
Copy link
Contributor

What does this PR do? A clear and concise description with your runway ticket url.
This PR resolved the issue of when devs have a local server running for Playbook and make a change to a React file (or SCSS file), the Playbook website will refresh several times in a row.

Screenshots: Screenshots to visualize your addition/change

How to test? Steps to confirm the desired behavior:

  1. Start local server
  2. Make a change to a React or SCSS file
  3. Observe only one reload

Checklist:

  • LABELS Add a label: enhancement, bug, improvement, new kit, deprecated, or breaking. See Changelog & Labels for details.
  • DEPLOY I have added the milano label to show I'm ready for a review.
  • TESTS I have added test coverage to my code.

@cmhorsey cmhorsey added milano 20 MAX - Deploy this PR to a review environment via Milano minor Semver Target labels Feb 26, 2025
@cmhorsey cmhorsey self-assigned this Feb 26, 2025
@cmhorsey cmhorsey requested a review from a team as a code owner February 26, 2025 13:57
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
milano 20 MAX - Deploy this PR to a review environment via Milano minor Semver Target
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant