Skip to content

Fix Newsletter's TransportBuilderTest#2579

Closed
mmenozzi wants to merge 1 commit intomagento:developfrom
mmenozzi:patch-1
Closed

Fix Newsletter's TransportBuilderTest#2579
mmenozzi wants to merge 1 commit intomagento:developfrom
mmenozzi:patch-1

Conversation

@mmenozzi
Copy link
Copy Markdown
Contributor

@mmenozzi mmenozzi commented Dec 2, 2015

Method declaration \Magento\Newsletter\Test\Unit\Model\Queue\TransportBuilderTest::testGetTransport() is now compatible with its parent \Magento\Framework\Mail\Test\Unit\Template\TransportBuilderTest::testGetTransport().

Method declaration `\Magento\Newsletter\Test\Unit\Model\Queue\TransportBuilderTest::testGetTransport()` is now compatible with its parent `\Magento\Framework\Mail\Test\Unit\Template\TransportBuilderTest::testGetTransport()`.
@davidalger
Copy link
Copy Markdown
Member

@mmenozzi Thanks for your contribution. Would you please accept the license/CLA?

@mmenozzi
Copy link
Copy Markdown
Contributor Author

mmenozzi commented Dec 2, 2015

Hi @davidalger, ok now it's signed.

@mmenozzi
Copy link
Copy Markdown
Contributor Author

mmenozzi commented Dec 2, 2015

I also think that this PR could fix the same error stated in #2365

@vkublytskyi
Copy link
Copy Markdown

@mmenozzi Thank you for your contribution but this PR could not be processed as initial issue already resalved in another commit

@mmenozzi
Copy link
Copy Markdown
Contributor Author

Yes it's true but my PR was 20 days before that commit ;).
Feel free to close this.
Bye.

magento-engcom-team pushed a commit that referenced this pull request May 22, 2018
[EngCom] Public Pull Requests - 2.3-develop
deiserh added a commit to deiserh/magento2 that referenced this pull request Sep 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants