Skip to content

feat: add can_view_threads relation#13

Merged
vitramir merged 1 commit into
mainfrom
noa/issue-67
Apr 19, 2026
Merged

feat: add can_view_threads relation#13
vitramir merged 1 commit into
mainfrom
noa/issue-67

Conversation

@casey-brooks
Copy link
Copy Markdown
Contributor

Summary

  • add organization can_view_threads computed relation
  • cover can_view_threads checks in OpenFGA model tests

Testing

  • buf generate buf.build/agynio/api --path agynio/api/authorization/v1
  • go test ./...
  • go build ./...

Refs #67

@casey-brooks
Copy link
Copy Markdown
Contributor Author

Test & Lint Summary

  • buf generate buf.build/agynio/api --path agynio/api/authorization/v1
  • go vet ./...
  • go test ./...
    • passed: 1, failed: 0, skipped: 0
  • go build ./...

Lint: go vet ./... (no issues)

@casey-brooks
Copy link
Copy Markdown
Contributor Author

Test & Lint Summary

  • buf generate buf.build/agynio/api --path agynio/api/authorization/v1
  • go test ./...

Copy link
Copy Markdown

@noa-lucent noa-lucent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

  • can_view_threads relation is modeled consistently with existing org-scoped computed relations (admin from cluster).
  • Added model tests cover owner/admin allow + member deny.

@vitramir vitramir merged commit 656ea39 into main Apr 19, 2026
1 check passed
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.

3 participants