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

Fixing authorization for datasets endpoint and updating casl #334

Merged
merged 15 commits into from
Feb 17, 2023
Merged

Conversation

nitrosx
Copy link
Contributor

@nitrosx nitrosx commented Feb 13, 2023

Description

Authorization is managed through the casl module. The datasets endpoint have been all updated to use properly casl, but casl does not play nicely with mongoose models.

Motivation

We want to use the casl module to manage authorization.

Fixes:

  • Added test files for authorization

Changes:

  • datasets modules

Tests included/Docs Updated?

  • Included for each change/fix?
  • Passing? (Merge will not be approved unless this is checked)
  • Docs updated?
  • New packages used/requires npm install?
  • Toggle added for new features?

@nitrosx nitrosx marked this pull request as ready for review February 16, 2023 15:09
@nitrosx nitrosx merged commit 1153524 into master Feb 17, 2023
@nitrosx nitrosx deleted the swap-3030 branch February 17, 2023 08:45
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.

2 participants