Skip to content

current user should be dependency-injected #863

Description

@tomholub

todo after #657 is done

instead of services and providers getting their own "currentAccountEmail", we should pass it in arguments to the methods. That way we achieve two things:

  • the methods no longer have to throw just in case they miss current email (since it will be provided)
  • we can remove a group of "middlemen classes" that only add account email to methods that need it - unneeded complexity

Additionally, ViewControllers should also receive accountEmail in their parameters from the parent view.

Somewhere in root view or router or similar there should be one single place that decides current user email and passes it down.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions