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

fix: Added permissions validation context to api error #858

Merged
merged 4 commits into from
Oct 10, 2023

Conversation

josebui
Copy link
Contributor

@josebui josebui commented Oct 2, 2023

Description

  • Added permissions validation context for story map approve membership mutation
  • Make story map invitation approve mutation require authentication

Checklist

  • Corresponding issue has been opened
  • New tests added

Related Issues

Fixes techmatters/terraso-web-client#1196

@josebui josebui marked this pull request as ready for review October 3, 2023 22:21
@josebui josebui requested a review from paulschreiber October 3, 2023 22:21
@josebui josebui merged commit 44f44cb into main Oct 10, 2023
6 checks passed
@josebui josebui deleted the fix/story-map-membership-approve-context branch October 10, 2023 19:28
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.

Bug: user invite to story map failed when user logged in with the wrong account.
2 participants