-
Notifications
You must be signed in to change notification settings - Fork 1
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
Devcon RC1 #124
Open
cryptix
wants to merge
57
commits into
main
Choose a base branch
from
devcon-rc1
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Devcon RC1 #124
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
-addAddress + verify in KC manager -stateManager tests passing with shipping updates -use requestId to resolve events -setting shipping address to order in guest checkout test -adding test for handlePingRequest -use next env var for chain id -use shopPublicClient / leave modifier empty -adding commit/choosePayment to client test
* separate authenticate from connect in client * connect and authenticate
-onDisconnect using wagmi hook -multiple imgs displayed for product detail screen -getTokenInformation for test chains -using getBaseTokenInfo in components
- new design implementation - waiting for tx after adding relay - error handling for merchants connect - logging non errors in color grey - merchant connect set client connection to pending - RelayClient and StateManager module - updates to product detail + adding blockHash - adding tokenAddresses json - separate eventStreamProcessing and addRelayOwnerAddress - connect wallet with rainbowkit for cart checkout - sendTransaction and fixing guest refresh - adding more order listeners
- all client tests working - guestCheckout working - allows next to find modules - disable typechecker for now - some fixes to client type checking - fix CI, removed unused tsconfig files
- client package - stateManager typing - add signer private key to vectors - sm tests load the vectors - fix payee chainid compare - tests passing but timer leaks - addRelaysToKeycards
- fixed relative import in stateManager - exposed stream - fix all leaks - removed unneeded dependcies flake.nix: remove unused deps github: use npm:next build, to build frontend in attempt to fix frontend build failure
- testing: setupStateManagerStores - fix create-store - items > listings - TEMP HACK keyCard nonce - restore guestCheckout_test.ts - fix orderStore functions - fix stateManager tests - fix guestCheckout tests use different relay branch
- cleanup bad merge - remove and add selected qty in cart - set eventNonceCounter - out of stock error in checkout flow - adding keycardNonceStore to test files - close keycardNonceStore
cancel order when user exits out of checkout flow using committedOrderID in checkout flow
* basic frontend tests working * fix type errors * update CI to skip front-end tests, and to skip type checking in the testing step
* adding validation for shipping * wrap random bytes with privateKeyToAccount * render correct token icon * useRef for authenticating guest checkout keycard * delete eventlisteners * update event returns [type, Order] * updating sm test for order update event * consolidate static page. memory usage issue fixed * display not enough stock error * remove searchparams from products page * Missing commits ci test (#133) * try deno 2.0.5 * add browser level to package.json * add pollyfill for webstreams --------- Co-authored-by: wanderer <[email protected]>
* minimize prod build * fix decimal lookup on sepolia
sorting orders by timestamp getting order timestamp from event.timestamp save order.timestamp for each update smol cleanup
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Snapshot PR to target cleanup patches.