Skip to content

Add example poncho project#14

Merged
burmajam merged 1 commit into
mainfrom
example-app
Jun 13, 2026
Merged

Add example poncho project#14
burmajam merged 1 commit into
mainfrom
example-app

Conversation

@burmajam

Copy link
Copy Markdown
Contributor

No description provided.

@burmajam burmajam force-pushed the example-app branch 2 times, most recently from ad3e3f0 to 50b1716 Compare June 13, 2026 22:37
@burmajam burmajam requested a review from svrdlans June 13, 2026 22:38
@burmajam burmajam marked this pull request as ready for review June 13, 2026 22:38

@svrdlans svrdlans left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Oh go get away 🙂
(ma idi begaj)

} = msg
) do
from(a in Model, where: a.id == ^id)
|> Repo.update_all(inc: [balance: amount])

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

IIRC Repo.update_all doesn't write timestamps
all good for this example, just saying - in real scenario Repo.update would be preferrable

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

good point for real prod app!

alias Banking.Listeners.Services

# Denormalization services (dispatched by listener)
servicep :account_opened!, Denormalizer, :denormalize

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'd probably move discoverablr services above private ones :)

@burmajam burmajam merged commit 94f7526 into main Jun 13, 2026
1 check passed
@burmajam burmajam deleted the example-app branch June 13, 2026 23:25
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.

2 participants