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

Additions and fixes #4

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Additions and fixes #4

wants to merge 7 commits into from

Conversation

ismyhc
Copy link
Contributor

@ismyhc ismyhc commented Dec 22, 2022

Hey!

Ive made some additions to the library where I needed some extra things in my nostr client.

  • Fixed json encoding to include the option to ignore escaping slashes. This was needed to handle encryption.
  • Added some functionality to the KeyPair
    • Expose private key string
    • Add bech32 support
  • Added a verified function to Event which will verify the event by signature.

Let me know if you have any questions :)

@cnixbtc
Copy link
Owner

cnixbtc commented Jan 4, 2023

Hey @ismyhc! Thanks for the PR and sorry for the late reply. I'll have a look at your changes and will get back to you this week!

@ismyhc
Copy link
Contributor Author

ismyhc commented Feb 26, 2023

Hey @ismyhc! Thanks for the PR and sorry for the late reply. I'll have a look at your changes and will get back to you this week!

Hey, just wanted to check if you were able to look at these and merge them in. I added one more minor commit that fixed typo of recommentServer to recommendServer.

Thanks!
Jacob

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