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
Removing duplicate date formatting for listing expiry date in listing dashboard #10264
Removing duplicate date formatting for listing expiry date in listing dashboard #10264
Conversation
I did not fill in the category for listing. That is what shows up extra space in the above screenshot |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small suggestion, but otherwise good to go @Rafi993!
{' '} | ||
| Expires on: <DateTime dateTime={expiryDate} /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
{' '} | |
| Expires on: <DateTime dateTime={expiryDate} /> | |
{' | Expires on: '}<DateTime dateTime={expiryDate} /> |
@@ -1,5 +1,6 @@ | |||
import PropTypes from 'prop-types'; | |||
import { h } from 'preact'; | |||
import { h, Fragment } from 'preact'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! I'm reacquainting myself with React (and learning more about Preact!) and I hadn't heard of Fragment
components before! Pretty neat; thanks for teaching me about them by using them in your PR 😸
Thanks @rhymes |
What type of PR is this?
Description
It removes duplicate date formatting code for listing expiry date in listing dashboard
QA Instructions, Screenshots, Recordings
Added tests?
Added to documentation?