Disallow deregistering/deqeueing of certain assets that break QM
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 35/100
Direção de pesquisa
Comece revisando os casos solicitados de wp_deregister_style e wp_dequeue_script e as referências vinculadas do Query Monitor em dispatchers/Html.php para os handles de assets necessários. Defina o comportamento do sniff de modo que dashicons e jquery sejam reportados quando forem deregistrados ou removidos da fila, enquanto handles com nomes semelhantes, como not_dashicons e not_jquery, não sejam; a conclusão exige cobertura tanto para exemplos de violação quanto de não violação.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
What problem would the enhancement address for VIP?
Sometimes certain core assets are deregistered/dequeued which break Query Monitor, which adds an extra step during debugging to track that down.
Dependencies are:
dashiconshttps://github.com/Automattic/vip-go-mu-plugins-built/blob/d38a2b0e4a1be875302942783cfe49f7f343e226/query-monitor/dispatchers/Html.php#L187-L192jqueryhttps://github.com/Automattic/vip-go-mu-plugins-built/blob/d38a2b0e4a1be875302942783cfe49f7f343e226/query-monitor/dispatchers/Html.php#L170-L199
Describe the solution you'd like
It'd be good to have a sniff that warns for specific assets that we do not recommend removing.
What code should be reported as a violation?
wp_deregister_style( 'dashicons' );
wp_dequeue_script( 'jquery' );
What code should not be reported as a violation?
wp_dequeue_style( 'not_dashicons' );
wp_dequeue_script( 'not_jquery' );
- Linguagem predominante
- PHP
- Estrelas
- 261
- Forks
- 44
- Merge médio
- 3d 9h
- PRs com merge (30d)
- 10
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de Automattic/VIP-Coding-Standards
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 48/100
Automattic/VIP-Coding-Standards#876 ·
Mantenedores costumam responder em até 1 dia
-
Breaking Change Type: Maintenance
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 35/100
Automattic/VIP-Coding-Standards#849 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 45/100
Automattic/VIP-Coding-Standards#832 ·
Mantenedores costumam responder em até 1 dia
-
WordPress.WP.AlternativeFunctions.file_system_operations error messages cannot be fixed anymoreAberta
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 48/100
Automattic/VIP-Coding-Standards#823 ·
Mantenedores costumam responder em até 1 dia
-
Status: Awaiting feedback
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 45/100
Automattic/VIP-Coding-Standards#754 · 3 comentários ·
Mantenedores costumam responder em até 1 dia
Todas as issues de Automattic/VIP-Coding-Standards
Issues semelhantes
-
C-CDA export: allergy concern act code, policy activity translation and empty id/@extension fail conformanceTalvez já em andamento @doctorman assumiu hoje. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
-
help wanted
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 82/100
-
scope:testing status:unconfirmed
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
wp-cli/wp-cli-tests#389 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 62/100
the-benchmarker/web-frameworks#10153 ·
Mantenedores costumam responder em até 1 dia