Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

feat!: add webp for all image styles #2618

Merged
merged 4 commits into from
Jan 20, 2022
Merged

feat!: add webp for all image styles #2618

merged 4 commits into from
Jan 20, 2022

Conversation

podarok
Copy link
Contributor

@podarok podarok commented Jan 19, 2022

Original Issue, this PR is going to fix: REPLACE WITH A LINK TO ISSUE ( publicly available )

Make sure these boxes are checked before asking for review of your pull request - thank you!

If there is a new feature or this is a bug fix - use 9.x-2.x branch. We'll tag for release if the bug is critical asap or tag for release next bug fix release until critical issue arrived.

Steps for review

            "  openy_med   8016                 hook_update_n   8016 - Apply Webp to all     ",
            "  ia_image                                         image styles.                ",
            "  openy_nod   8009                 hook_update_n   8009 - Apply Webp to all     ",
            "  e                                                image styles.                ",
            "  openy_nod   8015                 hook_update_n   8015 - Apply Webp to all     ",
            "  e_blog                                           image styles.                ",
            "  openy_nod   8017                 hook_update_n   8017 - Apply Webp to all     ",
            "  e_categor                                        image styles.                ",
            "  y                                                                             ",
            "  openy_nod   8003                 hook_update_n   8003 - Apply Webp to all     ",
            "  e_event                                          image styles.                ",
            "  openy_nod   8007                 hook_update_n   8007 - Apply Webp to all     ",
            "  e_news                                           image styles.                ",
            "  openy_nod   8015                 hook_update_n   8015 - Apply Webp to all     ",
            "  e_program                                        image styles.                ",
            "  openy_nod   8004                 hook_update_n   8004 - Apply Webp to all     ",
            "  e_social_                                        image styles.                ",
            "  post                                                                          ",
            "  openy_prg   8013                 hook_update_n   8013 - Apply Webp to all     ",
            "  f_banner                                         image styles.                ",
            "  openy_prg   8010                 hook_update_n   8010 - Apply Webp to all     ",
            "  f_gallery                                        image styles.                ",
            "  openy_prg   8006                 hook_update_n   8006 - Apply Webp to all     ",
            "  f_mbrshp_                                        image styles.                ",
            "  calc                                                                          ",
            "  openy_prg   8010                 hook_update_n   8010 - Apply Webp to all     ",
            "  f_small_b                                        image styles.                ",
            "  anner                                                                         ",
            "  openy_prg   8011                 hook_update_n   8011 - Apply Webp to all     ",
            "  f_teaser                                         image styles.                ",

General checks

Thank you for your contribution!

@podarok podarok added PR: Needs Work Unfinished task. Issues still there PR: DO NOT MERGE Work in Progress or something not to be merged at all Level: Middle Needs skilled developer to work on Status: Needs work Unfinished task. Issues still there Type: Enhancement Change request. New functionality. Severity: Normal Not critical, but still important. Selected for development. labels Jan 19, 2022
@podarok podarok self-assigned this Jan 19, 2022
@gundevel
Copy link
Collaborator

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build3700/install.php


Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/3700/

@gundevel
Copy link
Collaborator

Open Y Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build3700/

@gundevel
Copy link
Collaborator

Open Y Complete profile installed at http://profile.openy.cibox.tools/build3700/

@podarok podarok changed the title feat!: add webp for openy_popups feat!: add webp for all image styles Jan 20, 2022
@gundevel
Copy link
Collaborator

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build3701/install.php


Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/3701/

@gundevel
Copy link
Collaborator

Open Y Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build3701/

@gundevel
Copy link
Collaborator

Open Y Complete profile installed at http://profile.openy.cibox.tools/build3701/

@podarok podarok added dependencies Issues with dependencies PR: Tested Manually tested. Green light for merging and removed PR: Needs Work Unfinished task. Issues still there PR: DO NOT MERGE Work in Progress or something not to be merged at all Level: Middle Needs skilled developer to work on Status: Needs work Unfinished task. Issues still there labels Jan 20, 2022
@gundevel
Copy link
Collaborator

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build3702/install.php


Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/3702/

@gundevel
Copy link
Collaborator

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build3703/install.php


Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/3703/

@gundevel
Copy link
Collaborator

Open Y Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build3702/

@gundevel
Copy link
Collaborator

Open Y Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build3703/

@podarok podarok merged commit 8d2559a into 9.x-2.x Jan 20, 2022
@podarok podarok deleted the webp branch January 20, 2022 14:14
@podarok podarok added this to the User Experience milestone Jan 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dependencies Issues with dependencies PR: Tested Manually tested. Green light for merging Severity: Normal Not critical, but still important. Selected for development. Type: Enhancement Change request. New functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants