We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
In our code base we have several approaches how we include and add JavaScripts in the head.
To avoid duplicate code and improve maintainability, we should limit ourselves to one function injectScript() inside of delayed.js. f.e.
injectScript()
delayed.js
The best way to find all the different approaches is to search for the following terms:
head.append(script)
head.appendChild(script)
document.createElement('script')
...
When centralising the approach we should also align on a default for the script attributes like crossorigin, async etc.
crossorigin
async
The text was updated successfully, but these errors were encountered:
No branches or pull requests
In our code base we have several approaches how we include and add JavaScripts in the head.
To avoid duplicate code and improve maintainability, we should limit ourselves to one function
injectScript()inside ofdelayed.js. f.e.The best way to find all the different approaches is to search for the following terms:
head.append(script)head.appendChild(script)document.createElement('script')...When centralising the approach we should also align on a default for the script attributes like
crossorigin,asyncetc.The text was updated successfully, but these errors were encountered: