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

[PLAY-1627] Fix Height Prop for HtmlOptions Support #3873

Merged
merged 20 commits into from
Nov 8, 2024
Merged

Conversation

jasperfurniss
Copy link
Contributor

@jasperfurniss jasperfurniss commented Nov 1, 2024

What does this PR do? A clear and concise description with your runway ticket url.
https://runway.powerhrg.com/backlog_items/PLAY-1627

Screenshots: Screenshots to visualize your addition/change
The htmlOptions now works, even with the addition of our new height prop.

How to test? Steps to confirm the desired behavior:

  1. Go to a doc example for Card of Flex kit, and add a new height prop.
  2. Test one that also has our htmlOptions prop.

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.

@jasperfurniss jasperfurniss self-assigned this Nov 4, 2024
@jasperfurniss jasperfurniss added milano 20 MAX - Deploy this PR to a review environment via Milano bug Fixes to issues discovered in Playbook (USED IN CHANGELOG) alpha minor Semver Target labels Nov 4, 2024
Copy link

github-actions bot commented Nov 4, 2024

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 14.6.2.pre.alpha.PLAY15814348

Your Alpha for NPM is 14.6.2-alpha.PLAY15814348

@jasperfurniss jasperfurniss changed the title Play 1581 [PLAY-1627] Fix Height Prop for HtmlOptions Support Nov 4, 2024
@jasperfurniss jasperfurniss marked this pull request as ready for review November 5, 2024 20:46
@jasperfurniss jasperfurniss requested review from a team as code owners November 5, 2024 20:46
@jasperfurniss jasperfurniss added milano 20 MAX - Deploy this PR to a review environment via Milano and removed milano 20 MAX - Deploy this PR to a review environment via Milano labels Nov 5, 2024
@jasperfurniss jasperfurniss added alpha and removed alpha labels Nov 5, 2024
Copy link

github-actions bot commented Nov 5, 2024

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 14.6.2.pre.alpha.PLAY15814384

Your Alpha for NPM is 14.6.2-alpha.PLAY15814384

nidaqg
nidaqg previously approved these changes Nov 6, 2024
@nidaqg nidaqg added the Code Approved Approved by a Playbook Admin label Nov 6, 2024
@Amax1296 Amax1296 added the Product Approved pending technical review, OK to merge to master label Nov 7, 2024
@jasperfurniss jasperfurniss merged commit 3ee381a into master Nov 8, 2024
5 checks passed
@jasperfurniss jasperfurniss deleted the PLAY-1581 branch November 8, 2024 16:57
Copy link

github-actions bot commented Nov 8, 2024

You merged this pr to master branch:
- Ruby Gem: 14.7.0.pre.rc.19
- NPM: 14.7.0-rc.19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alpha bug Fixes to issues discovered in Playbook (USED IN CHANGELOG) Code Approved Approved by a Playbook Admin milano 20 MAX - Deploy this PR to a review environment via Milano minor Semver Target Product Approved pending technical review, OK to merge to master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants