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 : fixes various errors and clean some code #127

Merged
merged 1 commit into from
Jul 24, 2023

Conversation

Harshit933
Copy link
Collaborator

  • Handles exceptions like invalid invoice and rune not having permissions.
  • also cleans some code

- Handles exceptions like invalid invoice and rune not having permissions.
- also cleans some code
@netlify
Copy link

netlify bot commented Jul 23, 2023

Deploy Preview for clnapp ready!

Name Link
🔨 Latest commit 3173857
🔍 Latest deploy log https://app.netlify.com/sites/clnapp/deploys/64bd83fddb3dbc000704049e
😎 Deploy Preview https://deploy-preview-127--clnapp.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

import 'package:trash_component/components/global_components.dart';

class PopUp {
static void showPopUp(
Copy link
Member

Choose a reason for hiding this comment

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

nice!

Copy link
Member

@vincenzopalazzo vincenzopalazzo left a comment

Choose a reason for hiding this comment

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

LGTM

When you deleted more code that you added mean that you did a good job! Thanks

While testing on the web I found two bugs, and I reported

@vincenzopalazzo vincenzopalazzo merged commit e962980 into dart-lightning:main Jul 24, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants