-
Notifications
You must be signed in to change notification settings - Fork 8.1k
New issue
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
Remove HTMLAttrXRef and mentions for ko #13662
Conversation
Preview URLs (121 pages)
Flaws (4139)Note! 2 documents with no flaws that don't need to be listed. 🎉 URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
No first query argument or 'browser-compat' or 'spec-urls' front-matter value passed`
No first query argument or 'browser-compat' front-matter value passed`
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
External URLs (11)URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
|
This pull request has merge conflicts that must be resolved before it can be merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Based on the failed Action, line 362 in files/ko/learn/forms/how_to_structure_a_web_form/index.md
is causing the error, but it is unrelated to the PR and it seems the Korean translation may need an update to ensure coherence of the page. I am not sure how you would like for me to deal with this.
Please do let me know how we should proceed :)
This pull request has merge conflicts that must be resolved before it can be merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! I've gone ahead and fixed merge conflicts, and am going to merge this ASAP to prevent any future conflicts due to the formatting efforts. I've spot-checked the files and they all look good to me!
I'm also ignoring the build errors since they aren't caused by this PR!
Removes
{{HTMLAttrXRef}}
macros and its mentions in the docs for ko to help with #5603. I reviewed my changes a few times, so hopefully everything is good, but let me know if any fixes/improvements are needed :)