Skip to content

Commit

Permalink
Merge pull request #475 from josephschorr/clarify-check-warning
Browse files Browse the repository at this point in the history
Have the check warning only apply to relations, not permissions
  • Loading branch information
josephschorr authored Mar 10, 2022
2 parents 3295ba0 + 037c986 commit a569c5a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion internal/graph/check.go
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ func (cc *ConcurrentChecker) Check(ctx context.Context, req ValidatedCheckReques
var directFunc ReduceableCheckFunc

// TODO(jschorr): Turn into an error once v0 API has been removed.
if relation.GetTypeInformation() == nil {
if relation.GetTypeInformation() == nil && relation.GetUsersetRewrite() == nil {
log.Ctx(ctx).Warn().Str("relation", relation.Name).Msg("Found relation without type information. Please switch to using schema. This will be an error in the future!")
}

Expand Down

0 comments on commit a569c5a

Please sign in to comment.