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

Fix Escargot 0.6 compat and skip broken link checker #1788

Conversation

Toflar
Copy link
Member

@Toflar Toflar commented Jun 3, 2020

Q A
Fixed issues -
Docs PR or issue -

Fixed compat with Escargot 0.6 which also automatically adds tags for data attributes found on links.

There have been reports in the forums that users have links that the broken link checker doesn't need to check because it will take forever and it's actually irrelevant.

That's possible using these changes by turning your <a href="foobar">link</a> into <a href="foobar" data-skip-broken-link-checker>link</a>.

@Toflar Toflar added the bug label Jun 3, 2020
@Toflar Toflar added this to the 4.9 milestone Jun 3, 2020
@Toflar Toflar self-assigned this Jun 3, 2020
@Toflar Toflar requested a review from fritzmg June 3, 2020 11:51
@Toflar
Copy link
Member Author

Toflar commented Jun 3, 2020

Failing tests unrelated 🤷‍♂️

@fritzmg
Copy link
Contributor

fritzmg commented Jun 3, 2020

Can we may be shorten the tag to data-skip-link-checker? :)

@leofeyer
Copy link
Member

leofeyer commented Jun 3, 2020

Failing tests unrelated 🤷‍♂️

Fixed.

@Toflar
Copy link
Member Author

Toflar commented Jun 3, 2020

Can we may be shorten the tag to data-skip-link-checker? :)

I would like to keep the name consistent with the name of the subscriber if you don't mind :)

@Toflar Toflar force-pushed the fix/escargot-06-compat-and-skip-broken-link-checker branch from 14d4746 to fb0e836 Compare June 3, 2020 15:01
bytehead
bytehead previously approved these changes Jun 3, 2020
@Toflar
Copy link
Member Author

Toflar commented Jun 9, 2020

I've taken advantage of this PR to also add timing information to the debug log which is imho a bug too (a log without time information isn't a super useful log at times ;)).

@leofeyer leofeyer merged commit 35491e9 into contao:4.9 Jun 9, 2020
@leofeyer
Copy link
Member

leofeyer commented Jun 9, 2020

Thank you @Toflar.

@Toflar Toflar deleted the fix/escargot-06-compat-and-skip-broken-link-checker branch June 9, 2020 14:00
@leofeyer leofeyer changed the title Fix/escargot 06 compat and skip broken link checker Fix Escargot 0.6 compat and skip broken link checker Jul 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants