Skip to content
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.

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

fix lnurl channel #3046

Merged
merged 8 commits into from
Feb 23, 2024
Merged

fix lnurl channel #3046

merged 8 commits into from
Feb 23, 2024

Conversation

pavanjoshi914
Copy link
Contributor

show proper error messages

Describe the changes you have made in this PR

fix lnurl callback
proper error messages while doing requests

Type of change

(Remove other not matching type)

  • fix: Bug fix (non-breaking change which fixes an issue)

Checklist

  • Self-review of changed code
  • Manual testing
  • Added automated tests where applicable
  • Update Docs & Guides
  • For UI-related changes
  • Darkmode
  • Responsive layout

show proper error messages
Copy link
Contributor

@reneaaron reneaaron left a comment

Choose a reason for hiding this comment

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

Nice! 🚀

src/app/screens/LNURLChannel/index.tsx Show resolved Hide resolved
src/app/screens/LNURLChannel/index.tsx Outdated Show resolved Hide resolved
src/app/screens/LNURLChannel/index.tsx Outdated Show resolved Hide resolved
@pavanjoshi914
Copy link
Contributor Author

sry i just quickly coded it. need to improve the pr a bit. thanks for the review addressing it

@pavanjoshi914 pavanjoshi914 marked this pull request as draft February 17, 2024 13:23
@pavanjoshi914 pavanjoshi914 marked this pull request as ready for review February 19, 2024 07:25
@reneaaron reneaaron marked this pull request as draft February 19, 2024 14:58
@pavanjoshi914 pavanjoshi914 marked this pull request as ready for review February 20, 2024 10:33
@pavanjoshi914 pavanjoshi914 added the next-release To be included in the next release label Feb 20, 2024
Copy link
Contributor

@reneaaron reneaaron left a comment

Choose a reason for hiding this comment

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

One little thing to change.

Otherwise utACK

src/app/screens/LNURLChannel/index.tsx Show resolved Hide resolved
@pavanjoshi914
Copy link
Contributor Author

will merge this. i think functionality is complete now. if there are further improvements we need to do. i can do a follow up pr

@pavanjoshi914 pavanjoshi914 merged commit 49eedcc into master Feb 23, 2024
7 checks passed
@pavanjoshi914 pavanjoshi914 deleted the fix-lnurl-channel branch February 23, 2024 06:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
next-release To be included in the next release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants