Skip to content

Conversation

@jschaedl
Copy link
Contributor

@jschaedl jschaedl commented Apr 17, 2021

Q A
Branch? 5.x
Bug fix? no
New feature? no
Deprecations? no
Tickets -
License MIT
Doc PR -

Tests on #39353 were failing because of missing entries in the $classToServices array. They are now added and the list is ordered alphabetically.

Missing entries:

  • LightSmsTransportFactory
  • SmsBiurasTransportFactory
  • MessageBirdTransport

@jschaedl jschaedl changed the title [FrameworkBundle] Order transport services alphabetically [FrameworkBundle] Order transport factory services alphabetically Apr 17, 2021
@jschaedl jschaedl changed the title [FrameworkBundle] Order transport factory services alphabetically [FrameworkBundle] Add missing entries and order transport factory services alphabetically Apr 17, 2021
@jschaedl jschaedl force-pushed the fix-notifier-transport-list-in-frameworkbundle branch from 4ec5c03 to 99895a2 Compare April 17, 2021 07:11
@jschaedl jschaedl force-pushed the fix-notifier-transport-list-in-frameworkbundle branch from 99895a2 to bb2c195 Compare April 17, 2021 14:54
@carsonbot carsonbot changed the title [FrameworkBundle] Add missing entries and order transport factory services alphabetically [FrameworkBundle][Notifier] Add missing entries and order transport factory services alphabetically Apr 17, 2021
@jschaedl
Copy link
Contributor Author

Closed in favour of #40843

@jschaedl jschaedl closed this Apr 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants