simonw/datasette

sql-formatter javascript is not now working with CloudFlare rocketloader

Open

#1.969 aberto em 23 de dez. de 2022

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Python (625 forks)batch import
bughelp wantedjavascript

Métricas do repositório

Stars
 (8.637 stars)
Métricas de merge de PR
 (Mesclagem média 67d 23h) (19 fundiu PRs em 30d)

Description

This is probably not a bug with datasette, but I thought you might want to know, @simonw.

I noticed today that my CloudFlare proxied datasette instance lost the "Format SQL" option. I'm pretty sure it was there last week.

In the CloudFlare settings, if I turn off Rocket Loader, I get the "Format SQL" option back.

Rocket Loader works by asynchronously loading the javascript, so maybe there was a recent change that doesn't play well with the asynch loading?

I'm up to date with https://github.com/simonw/datasette/commit/e03aed00026cc2e59c09ca41f69a247e1a85cc89

Guia do colaborador

Direção de pesquisa
Investigue como o JavaScript do sql formatter é carregado no Datasette e por que falha com o CloudFlare Rocket Loader. Verifique se o script depende de carregamento síncrono ou de certa prontidão do DOM. Considere usar uma tag de script com 'defer' ou 'async' adequadamente, ou inicializar o formatador após o carregamento do script.
Pilha de tecnologia
pythonjavascript
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptCloudFlare
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65