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-343]- Date Pickr SCSS Bug Fix #2086

Merged
merged 1 commit into from Sep 23, 2022

Conversation

nidaqg
Copy link
Contributor

@nidaqg nidaqg commented Sep 22, 2022

Screens

What it looked like broken:

screen-shot-2022-09-21-at-8-48-59-am

What it looks like with fix:

Screen Shot 2022-09-22 at 9 07 07 AM

Breaking Changes

No breaking changes, fixes to SCSS that were breaking date pickr with time variant in Nitro

Runway Ticket URL

Runway Ticket

How to test this

Test in milano env as well as in nitro in dev_docs. Check to make sure 'Time Selection' version of date Pickr renders as expected and SCSS functions properly.

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

@nidaqg nidaqg added bug Fixes to issues discovered in Playbook (USED IN CHANGELOG) milano 20 MAX - Deploy this PR to a review environment via Milano labels Sep 22, 2022
@nidaqg nidaqg requested a review from a team as a code owner September 22, 2022 13:10
@nidaqg nidaqg self-assigned this Sep 22, 2022
@jasperfurniss jasperfurniss added needs alpha testing Ready for Release merged to master, ready for a versioned released labels Sep 22, 2022
@jasperfurniss jasperfurniss merged commit 756a233 into master Sep 23, 2022
@jasperfurniss jasperfurniss deleted the PLAY-343-Datepickr-Scss-Bug branch September 23, 2022 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes to issues discovered in Playbook (USED IN CHANGELOG) milano 20 MAX - Deploy this PR to a review environment via Milano needs alpha testing Ready for Release merged to master, ready for a versioned released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants