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

merkletree API #154

Closed
wants to merge 14 commits into from
Closed

merkletree API #154

wants to merge 14 commits into from

Conversation

tgrecojs
Copy link
Contributor

@tgrecojs tgrecojs commented Jan 3, 2025

The PR introduces the node.js API for obtaining claims.

tgrecojs and others added 14 commits December 12, 2024 20:23
…ng handled correctly when processed through updated offerHandler code
…itedness as generic interface for creating AsyncGenerator subscribtions (#150)

This commit contains test for:
- Handling bobs offer
- Subscribing to bobs wallet
- Verifying Alice's inability to make a second claim attempt
…xisting test code to properly evaluate assertions against updated makeClaimAirdropInvitation

- added Observable factory function for watching AsyncGenerators
- implemented logic for informing smart wallets of tribbles issuer (fixes issue where provisionSmartWallet fails when interfacing with a new issuer)
- added import statements for various type definitions (a practice that should continue whenever faced with VSCode intellisense errors stemming from unknown types)
…his commit contains code that was introduced to the codebase on a seperate branch**. This is meant to be as an acknowledgement of this undesired (yet somewhat necessary) process
* refactor(offerHandler):
	- moved offerHandler book keeping so that it takes place at the time when Payments are sent to claimant's depositFacets
	- removed instances where precomputed logic was being used once
	- removed address string from accountStore lookup message to eliminate unnecessary computation
- added test code demonstrating makePauseInvitation working as expected
- removed instances of unnecessary duplicate tests assertions
- renamed e2e.test.js to coreEval.test.js
elides properties such as mnenomic, pubkey.type, name, etc. as they are not needed for api testing
@tgrecojs tgrecojs closed this Jan 3, 2025
@tgrecojs tgrecojs deleted the api branch January 8, 2025 15:42
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.

1 participant