You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Some things that could be improved on after merging #622:
All remaining uses of ContextWrapper should be removed
Make @Inject in NUI screens use the game context. This could be quite a bit of work.
Document the roles of the service registry classes (CoreService, GameConfigurationService, SolGameServiceRegistry etc.)
Try to find a better alternative to EventReceiverServiceRegistry. It uses the older getSubtypesOf method, which might be better replaced with newer automatic techniques now provided by gestalt-di.
If ModuleManager initialisation fails, try to find a way of exiting gracefully whilst displaying an error.
Some things that could be improved on after merging #622:
ContextWrappershould be removed@Injectin NUI screens use the game context. This could be quite a bit of work.CoreService,GameConfigurationService,SolGameServiceRegistryetc.)EventReceiverServiceRegistry. It uses the oldergetSubtypesOfmethod, which might be better replaced with newer automatic techniques now provided by gestalt-di.ModuleManagerinitialisation fails, try to find a way of exiting gracefully whilst displaying an error.