Skip to content

Conversation

@ikripaka
Copy link
Collaborator

@ikripaka ikripaka commented Nov 5, 2025

No description provided.

@imaginator
Copy link
Collaborator

Unable to confirm merge. Seems a rebase is needed.

@ikripaka
Copy link
Collaborator Author

ikripaka commented Nov 5, 2025

Sure, I'll finish it tomorrow

@ikripaka ikripaka force-pushed the feature/nostr_interaction branch from d571097 to c62116c Compare November 6, 2025 07:20
chore:
* introduce helpers for parsing inner content of file
* add error handling for missing files and other errors (to understand what actually happened)
* add another function for getting events by its id
* change logger env variable name
chore:
* remove sending of anyhow error from library
* add example files for handling cli
* remove hardcoded key in test
* make optional by now parameters for sending oreders
@KyrylR KyrylR marked this pull request as draft November 7, 2025 09:35
@ikripaka ikripaka marked this pull request as ready for review November 7, 2025 14:19
@ikripaka ikripaka requested a review from KyrylR November 7, 2025 14:20
Copy link
Collaborator

@KyrylR KyrylR left a comment

Choose a reason for hiding this comment

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

LGTM

@ikripaka ikripaka changed the base branch from main to dev November 7, 2025 14:33
@ikripaka ikripaka merged commit 8757982 into dev Nov 7, 2025
1 check passed
@ikripaka ikripaka deleted the feature/nostr_interaction branch November 7, 2025 14:34
@ikripaka ikripaka changed the title WIP proof of concept on replies in nostr Proof of concept on replies in nostr Nov 11, 2025
zahorodnyi pushed a commit to zahorodnyi/simplicity-dex that referenced this pull request Nov 19, 2025
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.

4 participants