andrewdavey / andrewdavey/vogue

Vogue is incorrect in assuming it's the last script in scripts array

オープン
#31 コメント 4 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
JavaScript
スター
267
フォーク
36
PR マージ指標
30日以内にマージされた PR はありません

説明

I am getting errors like this:
"NetworkError: 404 NOT FOUND - http://127.0.0.1:8000/products/undefinedsocket.io/socket.io.js"

The problem lies in getScriptInfo() in vogue-client.js

it assumes the Vogue script will be the last one in the array returned by `document.getElementsByTagName("script")` but because it appends itself to the ... and common practice is now to put all your scripts at the bottom of the (see html5 boilerplate etc)... so it is not selecting the Vogue script, it's grabbing my Google Analytics script block instead.

It really needs to look for `vogue-client.js` specifically.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。