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

[PBNTR-523] Enable Kits Instead of Text for Radio label - React Only #3691

Merged
merged 8 commits into from
Sep 25, 2024

Conversation

nickamantia
Copy link
Contributor

@nickamantia nickamantia commented Sep 13, 2024

PBNTR-523

This PR adds the ability to pass in a kit as a child to the Radio kit. When the child is clicked, it will check off that Radio button.

Screenshot 2024-09-17 at 1 09 18 PM

How to test? Steps to confirm the desired behavior:

  1. Go to /kits/radio/react to confirm functionality.

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.

@nickamantia nickamantia self-assigned this Sep 13, 2024
Copy link

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 14.3.2.pre.alpha.pbntr523enablekitsforradiofix3809

Your Alpha for NPM is 14.3.2-alpha.pbntr523enablekitsforradiofix3809

@nickamantia nickamantia added the enhancement New Features, Props, & Variants (USED IN CHANGELOG) label Sep 18, 2024
@nickamantia nickamantia marked this pull request as ready for review September 18, 2024 17:17
@nickamantia nickamantia requested review from a team as code owners September 18, 2024 17:17
@nickamantia nickamantia added the milano 20 MAX - Deploy this PR to a review environment via Milano label Sep 19, 2024
@nickamantia nickamantia added alpha and removed alpha labels Sep 19, 2024
@nickamantia nickamantia added alpha and removed alpha labels Sep 19, 2024
Copy link

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 14.4.0.pre.alpha.pbntr523enablekitsforradiofix3825

Your Alpha for NPM is 14.4.0-alpha.pbntr523enablekitsforradiofix3825

Copy link

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 14.4.0.pre.alpha.pbntr523enablekitsforradiofix3826

Your Alpha for NPM is 14.4.0-alpha.pbntr523enablekitsforradiofix3826

@nidaqg nidaqg added Code Approved Approved by a Playbook Admin Product Approved pending technical review, OK to merge to master labels Sep 23, 2024
@nidaqg nidaqg added this pull request to the merge queue Sep 25, 2024
Merged via the queue into master with commit 15ed62b Sep 25, 2024
5 checks passed
@nidaqg nidaqg deleted the pbntr-523-enable-kits-for-radio-fix branch September 25, 2024 12:09
nidaqg pushed a commit that referenced this pull request Sep 26, 2024
**What does this PR do?** A clear and concise description with your
runway ticket url.
Copy the new URL from the original idea in Tempo [PR
#3691](powerhome/tempo#3691)

**Screenshots:** Screenshots to visualize your addition/change


**How to test?** Steps to confirm the desired behavior:
1. Go to a Milano environment
2. Click on "More" in the top right, and then "logs"
3. Logs won't appear because the URL needs to change


#### Checklist:
- [ ] **LABELS** Add a label: `enhancement`, `bug`, `improvement`, `new
kit`, `deprecated`, or `breaking`. See [Changelog &
Labels](https://github.com/powerhome/playbook/wiki/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.
nickamantia added a commit that referenced this pull request Oct 18, 2024
…3691)

[PBNTR-523](https://runway.powerhrg.com/backlog_items/PBNTR-523)

This PR adds the ability to pass in a kit as a child to the Radio kit.
When the child is clicked, it will check off that Radio button.


![Screenshot 2024-09-17 at 1 09 18
PM](https://github.com/user-attachments/assets/b91c5138-8704-4612-abf3-d4a990a02320)

**How to test?** Steps to confirm the desired behavior:
1. Go to `/kits/radio/react` to confirm functionality.


#### Checklist:
- [ ] **LABELS** Add a label: `enhancement`, `bug`, `improvement`, `new
kit`, `deprecated`, or `breaking`. See [Changelog &
Labels](https://github.com/powerhome/playbook/wiki/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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alpha Code Approved Approved by a Playbook Admin enhancement New Features, Props, & Variants (USED IN CHANGELOG) milano 20 MAX - Deploy this PR to a review environment via Milano Product Approved pending technical review, OK to merge to master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants