Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Speedreader] Failed to parse schema from @context in json+ld script. #27576

Closed
boocmp opened this issue Dec 30, 2022 · 1 comment · Fixed by brave/brave-core#16500
Closed

Comments

@boocmp
Copy link

boocmp commented Dec 30, 2022

Description

Steps to Reproduce

Open https://www.theverge.com/2022/12/28/23529705/amazon-drone-delivery-prime-air-california-texas in reader mode.

Actual result:

No publishing date in the article header.

Expected result:

Date displayed if it is presented in json+ld.

Reproduces how often:

Easy 100%.

@boocmp boocmp self-assigned this Dec 30, 2022
@rebron rebron added feature/speedreader priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes labels Dec 30, 2022
@brave-builds brave-builds added this to the 1.49.x - Nightly milestone Jan 16, 2023
@stephendonner
Copy link

Verified PASSED using

Brave 1.49.88 Chromium: 110.0.5481.77 (Official Build) beta (x86_64)
Revision 65ed616c6e8ee3fe0ad64fe83796c020644d42af-refs/branch-heads/5481@{#839}
OS macOS Version 11.7.3 (Build 20G1116)

(Reproduced in 1.48.157

Confirmed the date shows up now in 1.49.88, where it didn't before

1.48.157 1.49.88
Screen Shot 2023-02-07 at 1 07 42 AM Screen Shot 2023-02-07 at 1 08 49 AM

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants