Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Fix issue with preview panel close button not appearing for certain messages containing a URL #6871

Merged
merged 2 commits into from
Sep 27, 2021
Merged

Fix issue with preview panel close button not appearing for certain messages containing a URL #6871

merged 2 commits into from
Sep 27, 2021

Commits on Sep 27, 2021

  1. Extract logic to a function 

    For better readability of the call to useAsyncMemo()

Signed-off-by: Paulo Pinto <paulo.pinto@automattic.com>
    @psrpinto
    psrpinto committed Sep 27, 2021
    Configuration menu
    Copy the full SHA
    dabc13c View commit details
    Browse the repository at this point in the history

  2. Filter out invalid previews 

    The call to cli.getUrlPreview() might return an empty object ({}), which means
there is in fact no preview for that URL.

Signed-off-by: Paulo Pinto <paulo.pinto@automattic.com>
    @psrpinto
    psrpinto committed Sep 27, 2021
    Configuration menu
    Copy the full SHA
    432dd99 View commit details
    Browse the repository at this point in the history