Switch to Contracts in the Mail Affects

[bretto36]

Laravel 9 Introduced Contracts for lots of different areas, mail seems to have been missed.

Thanks for the great package!

[luceos]

Thank you so much!

[codecov]

Codecov Report

Merging #256 (fecd461) into 2.x (c850e05) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                2.x     #256   +/-   ##
=========================================
  Coverage     98.96%   98.96%           
  Complexity      290      290           
=========================================
  Files            98       98           
  Lines           776      776           
=========================================
  Hits            768      768           
  Misses            8        8           

[ArlonAntonius]

Saw the change in Laravel, makes sense to me.
Did you test it @bretto36 ?

[bretto36]

Yes. The tests ran. And when I had the error on my project changing this in the vendor folder manually allowed my tests to run. I can’t see it causing any issues as the previous class extends the contract classes now. On 22 Jun 2022, at 2:45 pm, Arlon Antonius ***@***.***> wrote:  @ArlonAntonius approved this pull request. Saw the change in Laravel, makes sense to me. Did you test it @bretto36<https://github.com/bretto36> ? — Reply to this email directly, view it on GitHub<#256 (review)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ABPOCCXOAIAVKVS4WX4GGALVQKK7XANCNFSM5ZHAFBFQ>. You are receiving this because you were mentioned.Message ID: ***@***.***>