Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[feat] User notifications backend #3280

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 15 commits into from
Jul 17, 2025
Merged

[feat] User notifications backend #3280

merged 15 commits into from
Jul 17, 2025

Conversation

@motorina0
Copy link
Collaborator

@motorina0 motorina0 commented Jul 17, 2025 •
edited
Loading

User Notifications Configs

Allow users to receive notifications when they send or receive a payment.
Polish the Account page.

image

User Nostr Message
image

@motorina0 motorina0 force-pushed the user_notifications_backend branch from c074392 to 949272d Compare July 17, 2025 11:14
@codecov
Copy link

codecov bot commented Jul 17, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 70.47619% with 31 lines in your changes missing coverage. Please review.

Project coverage is 57.50%. Comparing base (a36ab2d) to head (5d41704).
Report is 1 commits behind head on dev.

Files with missing lines Patch % Lines
lnbits/core/services/notifications.py 67.90% 26 Missing ⚠️
lnbits/core/services/funding_source.py 33.33% 2 Missing ⚠️
lnbits/core/tasks.py 50.00% 1 Missing ⚠️
lnbits/core/views/admin_api.py 50.00% 1 Missing ⚠️
lnbits/core/views/user_api.py 0.00% 1 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##              dev    #3280      +/-   ##
==========================================
+ Coverage   57.40%   57.50%   +0.10%     
==========================================
  Files         114      114              
  Lines       14284    14320      +36     
==========================================
+ Hits         8200     8235      +35     
- Misses       6084     6085       +1

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@motorina0 motorina0 requested review from arcbtc, dni and talvasconcelos and removed request for dni July 17, 2025 13:45
@motorina0 motorina0 marked this pull request as ready for review July 17, 2025 13:45
@dni dni force-pushed the user_notifications_backend branch from f98647a to 5d41704 Compare July 17, 2025 14:07
@dni dni mentioned this pull request Jul 17, 2025
Closed
@dni dni merged commit dce2bfb into dev Jul 17, 2025
14 checks passed
@dni dni deleted the user_notifications_backend branch July 17, 2025 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dni dni dni approved these changes

@arcbtc arcbtc Awaiting requested review from arcbtc

@talvasconcelos talvasconcelos Awaiting requested review from talvasconcelos

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@motorina0 @dni @arcbtc