Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 14 additions & 0 deletions comments/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,20 @@ sf dev
sf publish
```

## What `sf dev` cannot show you

`sf dev` brokers no platform services, so `addComment` stops at its first
`ctx.spam.check` call with `zero_spam_unavailable`. Guarding that one call is
enough to post a comment locally, because `ctx.email.send` sits behind
`if (from && notify)` and the two email variables are unset by default. Set
them and it fails the same way, with `zero_email_unavailable`.
`ctx.gravatar.avatarUrl` is fine offline: it is a hash and a query string, not
a request.

The local transaction boundary is also the whole invocation rather than the
`ctx.transaction()` block, so use a published Space to check that the comment
row and the email outbox row really commit together.

The mutation does not call `ctx.invalidate("comments")`. A successful table
write already causes live queries to refresh; explicit invalidation only narrows
which named subscriptions refresh.
Expand Down
Loading