Skip to content

Follow-ups for Gestalt-DI #659

@BenjaminAmos

Description

@BenjaminAmos

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.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions