feat: createOrUpdatePeerByUrl mutation #1857
Merged
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
# Conflicts: # pnpm-lock.yaml
# Conflicts: # pnpm-lock.yaml
…ng-mutation # Conflicts: # postman/collections/Interledger.json
dragosp1011
previously approved these changes
Sep 18, 2023
mkurapov
changed the title
mkurapov
dismissed stale reviews from dragosp1011 and njlie
The base branch was changed.
# Conflicts: # packages/backend/src/auto-peering/errors.ts # packages/backend/src/auto-peering/routes.test.ts # packages/backend/src/auto-peering/routes.ts # packages/backend/src/auto-peering/service.test.ts # packages/backend/src/auto-peering/service.ts
✅ Deploy Preview for brilliant-pasca-3e80ec canceled.
|
beniaminmunteanu
pushed a commit
that referenced
this pull request
Sep 29, 2023
* feat(backend): WIP - add feeid to quote table * refactor(backend): rename quote gql field * fix(backend): add fee relation * feat(backend): WIP calculate fees in quote create * refactor(backend): rename gql quote field * fix(backend): rm unused with graph fetched * feat(backend): move fee calculation to quote service * fix(backend): use quote error * fix(backend): rm unused fn * refactor(backend): rename quote send amount value * refactor(backend): rename quote send amount to debit amount * refactor(backend): rename out payment send amount to debit * chore(backend): format * chore(documentation): update gql docs * fix(backend,localenv): trx amount * fix(backend): handle cross currency fee in quote * refactor: update spec and rename more fields * fix(backend): outgoing limit test * refactor(backend): rename limits field * rename sendAmount to debitAmount * fix(localenv): postman ecommerce flow * fix(backend): remove useless checks * chore(auth): update open-payments package version * chore(backend): rm commented out code * fix(auth): migration to rename jsonb col * chore(auth): format * feat: add DuplicatePeer error * fix: pass in peerService to autoPeeringService * chore: update diff * chore: add to peer resolver test * feat: accept peering request * fix(backend): remove asset join * refactor(backend): remove variable assignment * feat: wire up router for auto peering * fix: update auto peering service * feat: correct config * config: update auto peering routes & errors * chore: prettier * feat: add createPeerByUrl mutation to schema * feat: add initiatePeeringRequest method to auto peering service * feat: add createPeerByUrl resolver * chore: fix lint * chore: add generate graphql files * chore: update graphql schema comment * fix: add type to /post peer error * feat: add postman request * chore: address comments * Update packages/backend/src/auto-peering/errors.ts Co-authored-by: Radu-Cristian Popa <radu.popa@breakpointit.eu> * chore: remove axios from app * chore: add axios to app * chore: fix whitespace * feature(backend): change error * reactor(documentation): update fee explanation * feat: use assetId in getByDestinationAddress * feat: update peer on duplicate peering request * chore: remove quote url references * feat: use proper outgoing/incoming http tokens * feat: update error during auto-peering request * fix(openapi): missing server warning * chore: fix route tests * feat: support maxPacketAmount and name in peering request * chore: add fields to peering request * feat: support maxPacketAmount and name in peering request * feat: default to peer given name if no name provided * chore: lint * fix: remove getter * Temporary fix * Revert "Temporary fix" This reverts commit bc7c69a. * chore(token-introspection): generate types * chore: update tests * refactor(documentation): rename sendAmount to debitAmount * Revert "Merge branch 'bc/1639/add-fees-to-quote' into mk/1802/accept-peering-request" This reverts commit 8fb0154, reversing changes made to 9619132. * ci: test heap bump * ci: logheapusage * chore: bump jest * chore: update tests * chore: update tests * Revert "ci: logheapusage" This reverts commit 66293e5. * revert fix * fix: revert pnpm lock * Update mutation request * feat: support (new) liquidityThreshold param * chore: rename to CreateOrUpdatePeerByUrlInput * feat: update peer on duplicate request * feat: update auto-peering resolver * Update mutation --------- Co-authored-by: Blair Currey <12960453+BlairCurrey@users.noreply.github.com> Co-authored-by: Radu-Cristian Popa <radu.popa@breakpointit.eu>
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.
Changes proposed in this pull request
createPeerOrUpdateByUrlmutationpeerUrl) to get peering details (AddPOST /peerroute handler #1802), and creates the peer locallyContext
Steps 1, 2, 5 of the diagram:
Fixes #1765
Checklist
fixes #number