[🐴] Reduce amount that message sent date is shown #4228
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
mozzius
commented
May 27, 2024
•
mozzius
commented
| Screenshot 1 | Screenshot 2 |
 |
 |
|
Your Render PR Server URL is https://social-app-pr-4228.onrender.com. Follow its progress at https://dashboard.render.com/web/srv-cpa2v2a0si5c739ucv3g. |
mozzius
changed the title
|
mozzius
force-pushed
the
samuel/reduce-meta
branch
from
8ec34a8 to
ebde83f
Compare
mozzius
force-pushed
the
samuel/reduce-meta
branch
from
65862ef to
20b4dee
Compare
mozzius
force-pushed
the
samuel/reduce-meta
branch
from
c296748 to
4189533
Compare
src/components/dms/DateDivider.tsx
Outdated
| } else if (localDateString(yesterday) === localDateString(timestamp)) { | ||
| date = _(msg`Yesterday`) | ||
| } else { | ||
| if (timestamp > oneWeekAgo) { |
There was a problem hiding this comment.
All my dates are coming back with just weekdays, even old chats
There was a problem hiding this comment.
Whoops, got the sign the wrong way around
src/components/dms/DateDivider.tsx
Outdated
Comment on lines
38
to
42
| const today = new Date() | ||
| const yesterday = new Date() | ||
| yesterday.setDate(today.getDate() - 1) | ||
| const oneWeekAgo = new Date() | ||
| oneWeekAgo.setDate(today.getDate() - 7) |
There was a problem hiding this comment.
Nit: we've got date-fns in here and that has some utils for this if it's any use. Peep useTimeAgo.test.ts
There was a problem hiding this comment.
Ah super helpful, thanks!
estrattonbailey
approved these changes
Oct 3, 2024
tkusano
pushed a commit
to tkusano/social-app
that referenced
this pull request
Oct 4, 2024
tkusano
added a commit
to tkusano/social-app
that referenced
this pull request
Oct 4, 2024
…cial#4228)" This reverts commit af8e342.
estrattonbailey
added a commit
that referenced
this pull request
Oct 7, 2024
* Update translation * Fixed * Update translation * Update translation * Update translation * Update translation * Suggested fix by Hima-Zinn * Update translation * Update translation * Update translation * Translate "pinned" * Update translation * Refactor video uploads (#5570) * Remove unused video field * Stop exposing video dispatch * Move cancellation out of the reducer * Make useUploadStatusQuery controlled by jobId * Rename SetStatus to SetProcessing This action only has one callsite and it's always passing "processing". * Move jobId into video reducer state * Make cancellation scoped * Inline useCompressVideoMutation * Move processVideo down the file * Extract getErrorMessage * useServiceAuthToken -> getServiceAuthToken * useVideoAgent -> createVideoAgent * useVideoUploadLimits -> getVideoUploadLimits * useUploadVideoMutation -> uploadVideo * Use async/await in processVideo * Inline onVideoCompressed into processVideo * Use async/await for uploadVideo * Factor out error messages * Guard dispatch with signal This lets us remove the scattered signal checks around dispatch. * Move job polling out of RQ * Handle poll failures * Remove unnecessary guards * Slightly more accurate condition * Move initVideoUri handling out of the hook * Remove dead argument It wasn't being used before either. * Remove unused detailed status This isn't being used because we're only respecting that state variable when isProcessing=true, but isProcessing is always false during video upload. If we want to re-add this later, it should really just be derived from the reducer state. * Harden the video reducer * Tie all spawned work to a signal * Preserve asset/media for nicer error state * Rename actions to match states * Inline useUploadVideo This abstraction is getting in the way of some future work. * Move MIME check to the only place that handles it * Manage video reducer from composer reducer (#5573) * Move video state into composer state * Represent video as embed This is slightly broken. In particular, we can't remove video yet because there's no action that results in video embed being removed. * Properly represent video as embed This aligns the video state lifetime with the embed lifetime. Video can now be properly added and removed. * Disable Add Video when we have images * Ignore empty image pick * Rename some files and variables (#5587) * Move composer reducers together * videoUploadState -> videoState * Inline videoDispatch * 🪵📌 (#5594) * [🐴] Reduce amount that message sent date is shown (#4228) * Fix profile header buttons (#5558) * Fix profile header buttons * Adjust labeler buttons too * Fix load state jumps * Small tweak for web * Remove log * Update * Revert "Update" This reverts commit b073c04. * Revert "Fix profile header buttons (#5558)" This reverts commit c00c5fa. * Revert "[🐴] Reduce amount that message sent date is shown (#4228)" This reverts commit af8e342. * Revert "🪵📌 (#5594)" This reverts commit 3ab5190. * Revert "Rename some files and variables (#5587)" This reverts commit 475708e. * Revert "Manage video reducer from composer reducer (#5573)" This reverts commit 59589e3. * Revert "Refactor video uploads (#5570)" This reverts commit f74f9bc. * Update translation * Update translation * Delete unused entries * Restored the trailing dot by #5622 --------- Co-authored-by: dan <[email protected]> Co-authored-by: Samuel Newman <[email protected]> Co-authored-by: Eric Bailey <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.