Skip to content

Update upgrade guide to reflect new image API#2341

Merged
gchtr merged 4 commits into2.x-docs-apifrom
2.x-image-docs
Oct 18, 2020
Merged

Update upgrade guide to reflect new image API#2341
gchtr merged 4 commits into2.x-docs-apifrom
2.x-image-docs

Conversation

@acobster
Copy link
Copy Markdown
Collaborator

Actual implementation to be done in #2333

Just some updates about the image/posts API that surfaced in #2316.

Actual implementation to be done in #2333
@acobster acobster requested a review from gchtr as a code owner October 15, 2020 18:29
@acobster
Copy link
Copy Markdown
Collaborator Author

see also: #2342

Copy link
Copy Markdown
Member

@gchtr gchtr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good. I fixed a small issue, because there never was a TimberImage() method on Timber\Post.

And then I realized the introductory wording for the updated Twig methods was off, too.

With all the changes coming in from the factories branch, I think it’s best I recheck the whole Upgrade Guide again, because updates were scattered in pull request against 2.x and against 2.x-docs-api.

I’ll do that separately.

@gchtr gchtr merged commit 72372e9 into 2.x-docs-api Oct 18, 2020
@gchtr gchtr deleted the 2.x-image-docs branch October 18, 2020 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants