-
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
[EASI-4636] GRB discussions card #2889
[EASI-4636] GRB discussions card #2889
Conversation
…i' into EASI-4636/discussions-response-component
…ssions-response-component
…ssions-response-component
Reverting this to draft to address changes now that #2882 is merged. |
Snapshot would always be outdated because of "X days ago" text within component
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.
Thanks for making these changes! Things look good from local testing, so I'm happy to just get this merged and we can loop back on if there's any more changes with seed/real API data that we end up needing to make, but from what I can see here things are all set!
5d4ca36
into
feature/EASI-4614_grb_discussions
EASI-4636
Figma
Figma with empty states
Description
<DiscussionPost>
component displays single discussion postFuture changes for followup PR once work for EASI-4634 is completed:
MentionTextArea
props are finalized before making these changes.How to test this change
PR Author Checklist
PR Reviewer Guidelines