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.

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

Development #54

Merged
merged 34 commits into from
Sep 4, 2023
Merged

Development #54

merged 34 commits into from
Sep 4, 2023

Conversation

mike-pisman
Copy link
Member

Description:

A small update mainly targeting CI. Added few small features and cleaned up the code.

Updates:

  • Resolved Mypy issue with BaseSettings class.
    For now the Dataclass derivatives cannot pass the type check, so the declaration of Settings class has to be ignored
  • Fixed code to pass all unit tests
  • Cleaned up files to pass Linting check
  • Resolved issue with setuptools-scm by manualy upgrading git
  • Moved all the project settings to pyproject.toml
  • Added query params to GET Group request
  • Added the enpoints to get all possible permissions
  • Added Coveralls action

Added open_router from Group routes to the app
Created route endpoints and actions for getting Groups and Workspace
permissions list

Closes #50
Moved action to get all permissions to a separate action file, so
getting list of permissions does not require a permission
Renamed get_all_group_policies action to get_group_policies
Renamed get_all_group_policies action to get_group_policies
Renamed get_all_group_policies action to get_group_policies
Expanded GET workspace request descriptions, renamed Workspace_resources to query_params
Added query param include, to get policies and/or members of the groups, when calling GET Group request
Deleted pytest.ini, the config was moved to pyproject.toml
Fixed issue with mypy not parsing BaseSettings type
@mike-pisman mike-pisman merged commit b134f05 into main Sep 4, 2023
2 checks passed
@mike-pisman mike-pisman deleted the development branch September 20, 2023 00:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant