Skip to content
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

Replace wrong single and double quotations with correct one #2977

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

Stefan-Stojanovic
Copy link
Collaborator

@Stefan-Stojanovic Stefan-Stojanovic commented Jan 4, 2022

Our "Solve this question" would break for type-in-the-gap with “, ”, and ’.

After testing, switching them to " and ' solves the issue.

It is hard to check only questions for all 273 files containing these "wrong quotations". So I did a content-wide search and replaced them in both content and questions/question answers.

I can't upload the video here as it is "too large". Msg me and I will send you the thread with the reasons and explanations with proof of why I made these changes.

@enki-hq
Copy link
Collaborator

enki-hq commented Jan 4, 2022

Insight

Insight Name Error Message
advanced-google-analytics
  • Google Analytics Top Tips pointing to https://neilpatel.com/blog/google-analytics-tips/ is broken because HTTP_403
don-t-repeat-yourself-dry-principle
  • www.thejavageek.com pointing to http://www.thejavageek.com/2015/04/10/dont-repeat-yourself-principle/ is broken because HTTP_404
what-is-the-blockchain Link checking took too long. Verify it manually.

Changelog

File Name Error Message
Pull Request Couldn't find a CHANGELOG entry for this pull request.

If you notice anything wrong with these rules contact [email protected].

@enki-hq
Copy link
Collaborator

enki-hq commented Jan 4, 2022

Insight

Insight Name Error Message
advanced-google-analytics
  • Google Analytics Top Tips pointing to https://neilpatel.com/blog/google-analytics-tips/ is broken because HTTP_403
don-t-repeat-yourself-dry-principle
  • www.thejavageek.com pointing to http://www.thejavageek.com/2015/04/10/dont-repeat-yourself-principle/ is broken because HTTP_404

Changelog

File Name Error Message
Pull Request Couldn't find a CHANGELOG entry for this pull request.

If you notice anything wrong with these rules contact [email protected].

@Stefan-Stojanovic
Copy link
Collaborator Author

The http://www.thejavageek.com/2015/04/10/dont-repeat-yourself-principle/ is broken as thejavageek.com domain is for sale so I will be replacing that link with something else

@enki-hq
Copy link
Collaborator

enki-hq commented Jan 4, 2022

Insight

Insight Name Error Message
advanced-google-analytics
  • Google Analytics Top Tips pointing to https://neilpatel.com/blog/google-analytics-tips/ is broken because HTTP_403

Changelog

File Name Error Message
Pull Request Couldn't find a CHANGELOG entry for this pull request.

If you notice anything wrong with these rules contact [email protected].

@enki-hq
Copy link
Collaborator

enki-hq commented Jan 4, 2022

Insight

Insight Name Error Message
advanced-google-analytics
  • Google Analytics Top Tips pointing to https://neilpatel.com/blog/google-analytics-tips/ is broken because HTTP_403

If you notice anything wrong with these rules contact [email protected].

@enki-hq
Copy link
Collaborator

enki-hq commented Jan 4, 2022

Insight

Insight Name Error Message
advanced-google-analytics
  • Google Analytics Top Tips pointing to https://neilpatel.com/blog/google-analytics-tips/ is broken because HTTP_403

If you notice anything wrong with these rules contact [email protected].

nem035
nem035 previously approved these changes Jan 11, 2022
@enki-hq
Copy link
Collaborator

enki-hq commented Jan 25, 2022

Insight

Insight Name Error Message
advanced-google-analytics
  • Google Analytics Top Tips pointing to https://neilpatel.com/blog/google-analytics-tips/ is broken because HTTP_403
property-value-shorthand
  • Enhanced Object Literals pointing to http://www.benmvp.com/learning-es6-enhanced-object-literals/ is broken because HTTP_308
what-is-stored-in-a-commit Link checking took too long. Verify it manually.

If you notice anything wrong with these rules contact [email protected].

@enki-hq
Copy link
Collaborator

enki-hq commented Jan 31, 2022

Insight

Insight Name Error Message
property-value-shorthand
  • Enhanced Object Literals pointing to http://www.benmvp.com/learning-es6-enhanced-object-literals/ is broken because HTTP_308
intro-to-product-analytics
  • Product vs Marketing Analytics pointing to http://upflow.co/l/Yem9/blog/2020/05/27/the-difference-between-product-analytics-and-marketing-analytics-and-why-you-need-both is broken because BLC_UNKNOWN

If you notice anything wrong with these rules contact [email protected].

I could not find a valuable link and the current one is deemed "unsafe", so I will remove it
@enki-hq
Copy link
Collaborator

enki-hq commented Jan 31, 2022

Insight

Insight Name Error Message
property-value-shorthand
  • Enhanced Object Literals pointing to http://www.benmvp.com/learning-es6-enhanced-object-literals/ is broken because HTTP_308

If you notice anything wrong with these rules contact [email protected].

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants