Skip to content

Add reason invisible link in post#4768

Open
user12986714 wants to merge 2 commits intoCharcoal-SE:masterfrom
user12986714:patch-53
Open

Add reason invisible link in post#4768
user12986714 wants to merge 2 commits intoCharcoal-SE:masterfrom
user12986714:patch-53

Conversation

@user12986714
Copy link
Copy Markdown
Contributor

Creating this as a new reason for stripping code blocks.

@ghost
Copy link
Copy Markdown

ghost commented Sep 15, 2020

Want me to organise some testing for this?

@tripleee
Copy link
Copy Markdown
Member

Are there currently samples in metasmoke which would trigger this?

@user12986714
Copy link
Copy Markdown
Contributor Author

user12986714 commented Sep 17, 2020

@NobodyNada
Copy link
Copy Markdown
Member

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

@teward
Copy link
Copy Markdown
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
Copy Markdown

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