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

cleanup: Allow for policy files to not specify projects #1103

Merged
merged 4 commits into from
Oct 4, 2023
Merged

cleanup: Allow for policy files to not specify projects #1103

merged 4 commits into from
Oct 4, 2023

Commits on Oct 4, 2023

  1. cleanup: Allow for policy files to not specify projects 

    This makes it easier to create policies for self-registered users, since
they can simply do `medic policy create -f <path to file>` and mediator
will just adopt the default project for the user.

This also moves the policy validations to the `validations.go` file in our
API, since it makes things more predictable.

Finally, this removes the context validation, as it's not super useful and we anyway
check for this along the code-base.
    @JAORMX
    JAORMX committed Oct 4, 2023
    Configuration menu
    Copy the full SHA
    614599b View commit details
    Browse the repository at this point in the history

  2. cli: Add flag to policy create to overwrite the project for a policy

    @JAORMX
    JAORMX committed Oct 4, 2023
    Configuration menu
    Copy the full SHA
    ca594b0 View commit details
    Browse the repository at this point in the history

  3. add comment to validator function

    @JAORMX
    JAORMX committed Oct 4, 2023
    Configuration menu
    Copy the full SHA
    2d17b18 View commit details
    Browse the repository at this point in the history

  4. policy: Validate name explicitly 

    This also removes the context validation test.
    @JAORMX
    JAORMX committed Oct 4, 2023
    Configuration menu
    Copy the full SHA
    bc4caaa View commit details
    Browse the repository at this point in the history