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

Add reason invisible link in post #4768

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

Conversation

user12986714
Copy link
Contributor

Creating this as a new reason for stripping code blocks.

@ghost
Copy link

ghost commented Sep 15, 2020

Want me to organise some testing for this?

@tripleee
Copy link
Member

Are there currently samples in metasmoke which would trigger this?

@user12986714
Copy link
Contributor Author

user12986714 commented Sep 17, 2020

@NobodyNada
Copy link
Member

Just for reference: Thomas Ward is also working on an alternative implementation of this using BeautifulSoup to parse the HTML.

@teward
Copy link
Member

teward commented Oct 4, 2020

Just for reference: Thomas Ward is also working on an alternative implementation of this using BeautifulSoup to parse the HTML.

This is dependent entirely on Makyen and others finishing the Post class revisions and Rule revisions to enable raw_body tests. Then the zero-length-link mechanism will be able to be completed.

@stale stale bot added the status: stale label Nov 5, 2020
@stale
Copy link

stale bot commented Nov 6, 2020

This issue has been closed because it has had no recent activity. If this is still important, please add another comment and find someone with write permissions to reopen the issue. Thank you for your contributions.

@stale stale bot closed this Nov 6, 2020
@makyen makyen added the status: confirmed Confirmed as something that needs working on. label Nov 6, 2020
@makyen makyen reopened this Nov 6, 2020
@stale stale bot removed the status: stale label Nov 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: confirmed Confirmed as something that needs working on.
Development

Successfully merging this pull request may close these issues.

5 participants