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.

Sign up for GitHub

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

Jump to bottom

Remove redundant SerializerInterface #53

Closed
naydav opened this issue Jan 30, 2020 · 1 comment
Closed

Remove redundant SerializerInterface #53

naydav opened this issue Jan 30, 2020 · 1 comment
Labels
Component: Notifier Progress: good first issue Good for newcomers

Comments

@naydav
Copy link
Contributor

naydav commented Jan 30, 2020

  1. Need to remove \Magento\NotifierApi\Model\SerializerInterface and \Magento\Notifier\Model\Serializer
  2. Replace all occurrences with an \Magento\Framework\Serialize\SerializerInterface
@engcom-Kilo engcom-Kilo self-assigned this Feb 12, 2020
@engcom-Kilo engcom-Kilo mentioned this issue Feb 13, 2020
Merged
5 tasks
naydav added a commit that referenced this issue Mar 10, 2020
@naydav
Merge pull request #98 from /issues/53
a7c1efa 
Security package/issues/53 - Remove redundant SerializerInterface
@naydav
Copy link
Contributor Author

naydav commented Mar 10, 2020

#98

magento-devops-reposync-svc pushed a commit that referenced this issue Oct 6, 2021
@nathanjosiah
Merge pull request #53 from magento-cia/metadata-service
0cb2224 
ACDOG-38: Repo Metadata Service Onboarding: CIA
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Component: Notifier Progress: good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@naydav @engcom-Kilo