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.

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

[LazyImage] refactor to twig.runtime & support intervention/image 3 #1766

Merged
merged 1 commit into from Apr 23, 2024

Conversation

kbond
Copy link
Member

@kbond kbond commented Apr 22, 2024

Q A
Bug fix? no
New feature? yes
Issues Fix #1465
License MIT
  • Refactored the code to use twig.runtime
  • Support intervention/image 3

@carsonbot carsonbot added the Status: Needs Review Needs to be reviewed label Apr 22, 2024
@kbond kbond merged commit 80b36bc into symfony:2.x Apr 23, 2024
36 of 37 checks passed
kbond added a commit that referenced this pull request Apr 24, 2024
This PR was squashed before being merged into the 2.x branch.

Discussion
----------

[LazyImage] Re-add forgotten twig.runtime

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Issues        | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exist, explain below instead -->
| License       | MIT

<!--
Replace this notice by a description of your feature/bugfix.
This will help reviewers and should be a good start for the documentation.

Additionally (see https://symfony.com/releases):
 - Always add tests and ensure they pass.
 - For new features, provide some code snippets to help understand usage.
 - Features and deprecations must be submitted against branch main.
 - Changelog entry should follow https://symfony.com/doc/current/contributing/code/conventions.html#writing-a-changelog-entry
 - Never break backward compatibility (see https://symfony.com/bc).
-->

Hi!

It looks like the Twig Runtime has been removed in #1766 while processing my review, so let's add it back :)

Commits
-------

6f92d33 [LazyImage] Re-add forgotten twig.runtime
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Review Needs to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Lazy-Image] Support for intervention/image v3
3 participants