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

Rewrote afterSave of product and category image attributes #3301

Merged
merged 4 commits into from
Jul 16, 2023

Conversation

luigifab
Copy link
Collaborator

@luigifab luigifab commented Jun 2, 2023

Description

For catalog and category image attributes, this PR allow to:

Tested with PHP 8.0 and 8.2.

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All automated tests passed successfully (all builds are green)
  • Add yourself to contributors list

Allow to override extensions and models
@github-actions github-actions bot added the Component: Catalog Relates to Mage_Catalog label Jun 2, 2023
@luigifab luigifab changed the title Updated afterSave of product and category image attribute Updated afterSave of product and category image attributes Jun 15, 2023
@empiricompany
Copy link
Contributor

tested on php 7.4

@ADDISON74 ADDISON74 self-assigned this Jul 14, 2023
fballiano
fballiano previously approved these changes Jul 15, 2023
@fballiano fballiano requested review from empiricompany, ADDISON74 and elidrissidev and removed request for empiricompany July 16, 2023 17:43
@fballiano fballiano changed the title Updated afterSave of product and category image attributes Rewrote afterSave of product and category image attributes Jul 16, 2023
@fballiano fballiano merged commit eb5d155 into OpenMage:main Jul 16, 2023
29 checks passed
@luigifab luigifab deleted the saveattr branch September 20, 2023 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Catalog Relates to Mage_Catalog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants