ipfs/ipfs-desktop

Shareable links should get preloaded by default

Open

#1,699 opened on Oct 15, 2020

View on GitHub
 (1 comment) (1 reaction) (0 assignees)JavaScript (5,632 stars) (912 forks)batch import
P1effort/hoursexp/intermediatehelp wantedkind/enhancementstatus/ready

Description

When user takes a screenshot or imports files via OS integration, we copy a shareable link to clipboard. However, the content is not present on gateway node until someone requests it, and that leads to bad UX (https://github.com/ipfs-shipyard/ipfs-desktop/issues/1604#issuecomment-707274675).

IPFS Companion sends HTTP HEAD request to trigger data preload.

I believe Desktop should do the same by default, but there needs to be an option to disable preload, in case user wants to avoid non-IPFS traffic coming from their machine.

Contributor guide

Tech stack
javascriptnodejsshell
Domain
desktopperformance
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-2 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
mostly clear
Prerequisites
Understanding of IPFSBasic knowledge of HTTP requestsFamiliarity with IPFS Desktop codebase
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60
Research direction
Examine how IPFS Companion implements preloading via HTTP HEAD requests. Look at the current clipboard and shareable link handling in IPFS Desktop's source code, particularly in the main process. Implement a similar preload mechanism when a shareable link is copied, with a user setting to disable preload. Review issue #1604 and its comments for UX context.
Shareable links should get preloaded by default · ipfs/ipfs-desktop#1699 | Good First Issue