chore: Merge 4.66.1 into single-server#6773
Conversation
* Fix image preview crashing on large pdfs * Update snapshots * Update app/containers/message/Urls.tsx Co-authored-by: Noach Magedman <nmagedman@gmail.com> --------- Co-authored-by: Noach Magedman <nmagedman@gmail.com>
|
Caution Review failedThe pull request is closed. WalkthroughVersion bump from 4.66.0 to 4.66.1 across Android, iOS, and npm configurations. URL image validation logic in Urls.tsx refactored with memoized function and HTTP HEAD request verification. Changes
Sequence DiagramsequenceDiagram
participant Component as Urls Component
participant useCallback as getImageUrl<br/>(memoized)
participant Effect as useEffect
participant API as HTTP API
participant State as imageUrl State
Effect->>useCallback: Call with url params
useCallback->>useCallback: Compute image URL<br/>(url.image or url.url)
useCallback-->>Effect: Return _imageUrl
Effect->>Effect: Check _imageUrl exists
rect rgb(200, 220, 250)
Note over Effect,API: URL Validation Phase
Effect->>API: HEAD request<br/>to _imageUrl
API-->>Effect: Response with<br/>content-type
end
rect rgb(220, 250, 220)
Note over Effect,State: Conditional Update
Effect->>Effect: Verify content-type<br/>indicates image
Effect->>State: Update imageUrl<br/>with _imageUrl
end
Estimated code review effort🎯 2 (Simple) | ⏱️ ~20 minutes
Possibly related PRs
Poem
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro Disabled knowledge base sources:
⛔ Files ignored due to path filters (1)
📒 Files selected for processing (6)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Proposed changes
Issue(s)
How to test or reproduce
Screenshots
Types of changes
Checklist
Further comments
Summary by CodeRabbit
Chores
Bug Fixes