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-153] Hashtag Kit Convert to TS #1886

Merged
merged 10 commits into from
Jun 6, 2022

Conversation

dmcindoe
Copy link
Contributor

@dmcindoe dmcindoe commented May 9, 2022

Breaking Changes

No just converting file from JSX to TSX

Runway Ticket URL

(https://nitro.powerhrg.com/runway/backlog_items/PLAY-153)

How to test this

Make sure Kit Sill works as normal.

Checklist:

  • LABELS Add a label: enhancement, bug, improvement, new kit, deprecated, or breaking. See Changelog & Labels for details.
  • DEPLOY Please add the Milano label when you are ready for a review.
  • SCREENSHOT Please add a screen shot or two.
  • SPECS Please cover your changes with specs.
  • READ DOCS Please make sure you have read and understand the Playbook Release Process

dmcindoe and others added 6 commits May 3, 2022 14:27
* Removed flow and Renamed file

* Updated Types

* Removed JSX file extentions from imports

* Fix syntax error

* Fix tests

Co-authored-by: Erica Winne <[email protected]>
* Started Converting

* Rename file to TSX and Updated types

* Added/Joined & GlobalProps

* Merge Master

* Changed back default blank string

Co-authored-by: Erica Winne <[email protected]>
…1849)

* Rendered rough example in browser

* Section separator between props and values

* Added pills and continued to change styling

* Fixed width of cards

* Fixed flexitem alignment

* Added title and body kits

* Fixed more styling

* Changed out background kit to fix border radius

* Fixed border radius

* Fixed border radius on other templates

* Updated caption card

* Fixed title margins

* Corrected camelcase in pills

* Three templates for visual guidelines

* Fixed syntax error

* Fixed syntax again

* Testing client-side code example colorization

* Use Rouge to format code examples

* Template Complete

Co-authored-by: Stephen Marshall <[email protected]>
@dmcindoe dmcindoe self-assigned this May 9, 2022
@dmcindoe dmcindoe requested a review from a team as a code owner May 9, 2022 18:04
@dmcindoe dmcindoe added milano 20 MAX - Deploy this PR to a review environment via Milano and removed wip labels May 12, 2022
@thestephenmarshall thestephenmarshall changed the title [Play-153] hashtag kit convert to typescript [Play-153] Hashtag Kit Convert to TS May 18, 2022
@thestephenmarshall thestephenmarshall added the Ready for Release merged to master, ready for a versioned released label May 23, 2022
@thestephenmarshall thestephenmarshall removed the milano 20 MAX - Deploy this PR to a review environment via Milano label Jun 6, 2022
@thestephenmarshall thestephenmarshall merged commit 8889ac9 into master Jun 6, 2022
@thestephenmarshall thestephenmarshall deleted the Play-153-Hashtag-Kit-Convert-to-Typescript branch June 6, 2022 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready for Release merged to master, ready for a versioned released typescript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants