GoogleChromeLabs/quicklink

site: use the files from dist folder instead of the CDN

Open

#389 aberto em 16 de jun. de 2023

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)JavaScript (10.861 stars) (431 forks)batch import
docsenhancementgood first issuehelp wanted

Description

Not sure how to load the dist folder which is one folder up from site.

This should make sure PR previews use the latest code plus we should be able to add SRI hashes later (see #336).

Guia do colaborador

Pilha de tecnologia
javascripthtml
Domain
frontendbuild system
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
half day
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
needs investigation
Prerequisites
Understanding of project's build processFamiliarity with site folder
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60
Direção de pesquisa
Examine the `site/` directory and how it currently references the CDN files. Determine how to load the `dist/` folder from a relative path. Review existing build scripts (likely in package.json) to ensure the `dist/` folder is built. Consider integrating SRI hashes as mentioned in issue #336. Test via PR previews.