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.

Sign up for GitHub

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

Jump to bottom

[DomCrawler] Add argument $normalizeWhitespace to `Crawler::innerText… #17722

Closed
fabpot opened this issue Jan 11, 2023 · 2 comments
Closed

[DomCrawler] Add argument $normalizeWhitespace to `Crawler::innerText… #17722

fabpot opened this issue Jan 11, 2023 · 2 comments
Labels
DomCrawler hasPR A Pull Request has already been submitted for this issue.
Milestone
6.3

Comments

@fabpot
Copy link
Member

fabpot commented Jan 11, 2023

Q A
Feature PR symfony/symfony#48940
PR author(s) @otsch
Merged in 6.3

We created this issue to not forget to document this new feature. We would really appreciate if you can help us with this task. If you are not sure how to do it, please ask us and we will help you.

To fix this issue, please create a PR against the 6.3 branch in the symfony-docs repository.

Thank you! 😄
@fabpot fabpot added this to the 6.3 milestone Jan 11, 2023
@otsch otsch mentioned this issue Jan 12, 2023
Merged
javiereguiluz added a commit that referenced this issue Jan 13, 2023
@javiereguiluz
minor #17729 [DomCrawler] Adapt docs for Crawler::innerText() (otsch)
982c8d5 
This PR was merged into the 6.3 branch.

Discussion
----------

[DomCrawler] Adapt docs for `Crawler::innerText()`

According to the changes from symfony/symfony#48940
And as requested in #17722

Commits
-------

e5047ee Adapt docs for Crawler::innerText()
@otsch
Copy link
Contributor

otsch commented Jan 13, 2023

@fabpot docs change is merged: #17729

@javiereguiluz
Copy link
Member

Closing as fixed in #17729. Thanks.

@xabbuh xabbuh added the hasPR A Pull Request has already been submitted for this issue. label Mar 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
DomCrawler hasPR A Pull Request has already been submitted for this issue.
Projects
None yet
Milestone
6.3
Development

No branches or pull requests
5 participants
@fabpot @javiereguiluz @xabbuh @otsch @carsonbot