Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

refactor(core): Use injectable classes for db repositories (part-1) (no-changelog) #5953

Merged
merged 7 commits into from
Apr 12, 2023
Merged

refactor(core): Use injectable classes for db repositories (part-1) (no-changelog) #5953

merged 7 commits into from
Apr 12, 2023

Commits on Apr 11, 2023

  1. feat(core): Use concrete classed for db repositories, and make them i… 

    …njectable
    @netroy
    netroy committed Apr 11, 2023
    Configuration menu
    Copy the full SHA
    20f4449 View commit details
    Browse the repository at this point in the history

  2. move all role table related queries to the role repository

    @netroy
    netroy committed Apr 11, 2023
    Configuration menu
    Copy the full SHA
    14ca36f View commit details
    Browse the repository at this point in the history

  3. add some unit tests for RoleRepository

    @netroy
    netroy committed Apr 11, 2023
    Configuration menu
    Copy the full SHA
    0e3d9de View commit details
    Browse the repository at this point in the history

  4. make RoleService injectable, and add unit tests for it

    @netroy
    netroy committed Apr 11, 2023
    Configuration menu
    Copy the full SHA
    9192c15 View commit details
    Browse the repository at this point in the history

  5. use Db.getConnection() everywhere

    @netroy
    netroy committed Apr 11, 2023
    Configuration menu
    Copy the full SHA
    4ed829d View commit details
    Browse the repository at this point in the history

  6. initialize DB before telemetry

    @netroy
    netroy committed Apr 11, 2023
    Configuration menu
    Copy the full SHA
    702d075 View commit details
    Browse the repository at this point in the history

Commits on Apr 12, 2023

  1. Start connection before getting InternalHook instance from the container

    @RicardoE105
    RicardoE105 committed Apr 12, 2023
    Configuration menu
    Copy the full SHA
    b6e1031 View commit details
    Browse the repository at this point in the history