kosmos/akkounts
8
1
Fork 0
Code Issues 22 Pull Requests 1 Actions Packages Releases 12 Activity

Add Webhooks and XMPP notifications for incoming sats #79

Merged
raucao merged 10 commits from feature/webhooks into master 2023-01-13 04:33:02 +00:00
Conversation 1 Commits 10 Files Changed 17 +1 -1
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit b0c787bbc7 - Show all commits

2
app/controllers/webhooks_controller.rb
View File

@@ -11,7 +11,7 @@ class WebhooksController < ApplicationController
head :unprocessable_entity and return
end
user = User.find_by(ln_account: payload[:user_login])
user = User.find_by!(ln_account: payload[:user_login])
raucao marked this conversation as resolved Outdated
bumi commented 2023-01-13 00:09:22 +00:00
Outdated
Review
Copy Link

should not be the case, but this might be nil. maybe we should do a User.find_by!

should not be the case, but this might be nil. maybe we should do a `User.find_by!`
raucao commented 2023-01-13 04:25:21 +00:00
Outdated
Review
Copy Link

Good idea! Fails much cleaner and more expressively than when trying to access properties on nil later.

Good idea! Fails much cleaner and more expressively than when trying to access properties on `nil` later.
# TODO make configurable
notify_xmpp(user.address, payload[:amount], payload[:memo])