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

FEATURE: Introduce RoleId and RoleIds value objects #3415

Open
wants to merge 7 commits into
base: 9.0
Choose a base branch
from
Open

FEATURE: Introduce RoleId and RoleIds value objects #3415

wants to merge 7 commits into from

Commits on Nov 11, 2024

  1. FEATURE: Introduce RoleId and RoleIds value objects 

    Resolves: #3414
    @bwaidelich
    bwaidelich committed Nov 11, 2024
    Configuration menu
    Copy the full SHA
    1e61872 View commit details
    Browse the repository at this point in the history

  2. Cosmetics to satisfy linter

    @bwaidelich
    bwaidelich committed Nov 11, 2024
    Configuration menu
    Copy the full SHA
    bf67991 View commit details
    Browse the repository at this point in the history

  3. more cosmetics

    @bwaidelich
    bwaidelich committed Nov 11, 2024
    Configuration menu
    Copy the full SHA
    52f5e63 View commit details
    Browse the repository at this point in the history

Commits on Nov 13, 2024

  1. Tweak Role constructor to accept RoleId

    @bwaidelich
    bwaidelich committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    32bbd2d View commit details
    Browse the repository at this point in the history

  2. Use simple type in PolicyService::getRole() 

    to keep unit tests working
    @bwaidelich
    bwaidelich committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    073a2b0 View commit details
    Browse the repository at this point in the history

  3. Fix Role::__toString()

    @bwaidelich
    bwaidelich committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    4186614 View commit details
    Browse the repository at this point in the history

  4. Cosmetic tweak to satisfy linter

    @bwaidelich
    bwaidelich committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    40d6023 View commit details
    Browse the repository at this point in the history