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

βœ… metabase_permissions_graph acceptance test #66

Merged
merged 4 commits into from
Sep 7, 2024

Conversation

flovouin
Copy link
Owner

@flovouin flovouin commented Sep 7, 2024

πŸ“ Description of the PR

The title says it all. This is even more needed as there is a breaking change in the permissions graph API in Metabase version .50. This will be used to test the required change.

πŸ™ Related GitHub issue(s)

πŸ•°οΈ Commits

  • ⬆️ Upgrade dependencies
  • πŸ”§ Revert to Metabase 0.49 for tests
  • βœ… Test the metabase_permissions_graph resource
  • πŸ“ Remove compatibility with Metabase .50 for now

@flovouin flovouin added the chore label Sep 7, 2024
@flovouin flovouin merged commit 09fc2b0 into main Sep 7, 2024
1 check passed
@flovouin flovouin deleted the chore/acceptance-tests-permissions-graph branch September 7, 2024 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant