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

Correctly respond to application/ld+json requests, part 2 #779

Merged
merged 1 commit into from
Oct 3, 2023
Merged

Correctly respond to application/ld+json requests, part 2 #779

merged 1 commit into from
Oct 3, 2023

Conversation

thebaer
Copy link
Member

@thebaer thebaer commented Sep 26, 2023

This finishes the work started in #766, ensuring that canonical URLs of blogs and posts (not just at their API endpoints) respond correctly to application/ld+json;... requests.

Fully addresses issue #564

  • ☑ I have signed the CLA

@thebaer
Correctly respond to application/ld+json requests, part 2
2275a28 
This finishes the work started in #766, ensuring that requests to
canonical URLs of blogs and posts (not just at their API endpoints)
respond correctly to `application/ld+json;...` requests.

Fully addresses issue #564
@thebaer thebaer added this to the 0.15 milestone Sep 26, 2023
@thebaer thebaer mentioned this pull request Sep 26, 2023
Closed
@thebaer
Copy link
Member Author

thebaer commented Oct 3, 2023

Confirmed that this is working correctly on Write.as in #564 (comment). Merging now.

@thebaer thebaer merged commit 4aad033 into develop Oct 3, 2023
@thebaer thebaer deleted the fix-ld-json-response-2 branch October 3, 2023 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
0.15
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@thebaer