-
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
Log pinning/unpinning #5594
Merged
Merged
Log pinning/unpinning #5594
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
arcalinea
temporarily deployed
to
samuel/log-pin - social-app PR #5594
October 3, 2024 13:34 — with
Render
Destroyed
|
estrattonbailey
approved these changes
Oct 3, 2024
estrattonbailey
added a commit
that referenced
this pull request
Oct 3, 2024
estrattonbailey
added a commit
that referenced
this pull request
Oct 3, 2024
* origin/main: 🪵📌 (#5594) Rename some files and variables (#5587) Manage video reducer from composer reducer (#5573) Refactor video uploads (#5570) Delete messages NUX (#5574) move files around (#5576) Make reply prompt more subtle on desktop (#5569) Ignore built files (#5572) import order (#5571) Add suspense boundary in onboarding (#5556) Tweak verify email dialog (#5555) Add intent/verify-email to go server (#5554)
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
This reverts commit 3ab5190.
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.
🪵📌