Skip to content

[Forwardport] Attribute set save admin controller refactor#16041

Closed
hitesh-wagento wants to merge 4 commits intomagento:2.3-developfrom
hitesh-wagento:2.3-develop-PR-port-15990
Closed

[Forwardport] Attribute set save admin controller refactor#16041
hitesh-wagento wants to merge 4 commits intomagento:2.3-developfrom
hitesh-wagento:2.3-develop-PR-port-15990

Conversation

@hitesh-wagento
Copy link
Copy Markdown
Contributor

Original Pull Request

#15990
Description
Remove direct use of object manager for admin attribute set save controller using constructor based dependency injection.

Manual testing scenarios
Created a new attribute set in admin under Stores -> Attribute Set -> Add Attribute Set.
Edit an existing attribute set under Stores -> Attribute Set

@magento-engcom-team magento-engcom-team added Partner: Wagento Pull Request is created by partner Wagento partners-contribution Pull Request is created by Magento Partner Component: Catalog labels Jun 12, 2018
@ihor-sviziev ihor-sviziev self-assigned this Jun 12, 2018
@ihor-sviziev
Copy link
Copy Markdown
Contributor

Original PR was closed, instead was created new PR with all needed changes. #16217

I'm closing this PR as not actual anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component: Catalog Partner: Wagento Pull Request is created by partner Wagento partners-contribution Pull Request is created by Magento Partner Progress: reject

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants