-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Add backdated post indicator #6156
Conversation
|
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.
code lgtm but have not tested (dont really know how to test, can test if theres a way to!)
<Prompt.DescriptionText> | ||
<Trans> | ||
This post claims to have been created on{' '} | ||
<RNText style={[a.font_bold]}>{niceDate(i18n, createdAt)}</RNText>, |
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 q, why do we use RNText
here?
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.
Nested styles
Sorry for jumping in here, this looks like it might be a good way to test? :) "Test plan Replace post.indexedAt with the current date (or go back to before the final commit)" |
Backdated post indicator. Requires backend changes to actually work
Test plan
Replace
post.indexedAt
with the current date (or go back to before the final commit)Confirm posts older than 24 hours show the indicator
Confirm that the wording is good