-
Notifications
You must be signed in to change notification settings - Fork 0
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
trcl 2408 show good til and created at in ios app #11
trcl 2408 show good til and created at in ios app #11
Conversation
57d9cff
to
db1e93f
Compare
let text = dydxFormatter.shared.interval(time: Date(milliseconds: expiresAtMilliseconds)) | ||
return dydxOrderDetailsViewModel.Item(title: DataLocalizer.localize(path: "APP.TRADE.GOOD_TIL"), | ||
let text = dydxFormatter.shared.dateAndTime(date: Date(milliseconds: expiresAtMilliseconds)) | ||
return dydxOrderDetailsViewModel.Item(title: DataLocalizer.localize(path: "APP.TRADE.GOOD_TIL_TIME"), | ||
value: .string(text)) | ||
} | ||
return nil |
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.
I think it should be .none, same as timePlaced, so it shows a "-".
Web shows a "-" right now, and Yi mentioned it should be a "-" on Slack. I just modified the ticket description to reflect that.
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.
…-and-created-at-in-ios-app
Links (dYdX Internal Use Only)
Linear Ticket: TRCL-2408 : Show "good Til" and "created At" in iOS app
Description / Intuition
screenshots provided for long term orders (open)
Before/After Screenshots or Videos
Type of Change