This repository has been archived by the owner on Jan 13, 2025. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 919
Add createHttpTransportForSolanaRpc
function
#3196
Merged
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
🦋 Changeset detectedLatest commit: ac099d6 The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
This was referenced Sep 2, 2024
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @lorisleiva and the rest of your teammates on Graphite |
This was referenced Sep 2, 2024
lorisleiva
force-pushed
the
loris/is-solana-request
branch
from
September 2, 2024 15:43
fa57791
to
94e1ede
Compare
lorisleiva
force-pushed
the
loris/http-transport-for-solana-rpc
branch
from
September 2, 2024 15:43
63f1c2f
to
7d05aec
Compare
lorisleiva
force-pushed
the
loris/is-solana-request
branch
from
September 2, 2024 15:47
94e1ede
to
4d54dbb
Compare
lorisleiva
force-pushed
the
loris/http-transport-for-solana-rpc
branch
2 times, most recently
from
September 2, 2024 15:50
fbb79ae
to
f1f5511
Compare
lorisleiva
force-pushed
the
loris/is-solana-request
branch
from
September 3, 2024 21:25
4d54dbb
to
37e83c2
Compare
lorisleiva
force-pushed
the
loris/http-transport-for-solana-rpc
branch
from
September 3, 2024 21:25
f1f5511
to
1c162f6
Compare
lorisleiva
force-pushed
the
loris/is-solana-request
branch
from
September 3, 2024 21:29
37e83c2
to
d3a3d56
Compare
lorisleiva
force-pushed
the
loris/http-transport-for-solana-rpc
branch
from
September 3, 2024 21:30
1c162f6
to
ad1559a
Compare
lorisleiva
force-pushed
the
loris/is-solana-request
branch
from
September 3, 2024 21:36
d3a3d56
to
cdf7109
Compare
lorisleiva
force-pushed
the
loris/http-transport-for-solana-rpc
branch
from
September 3, 2024 21:36
ad1559a
to
674b13b
Compare
lorisleiva
force-pushed
the
loris/is-solana-request
branch
from
September 3, 2024 22:11
cdf7109
to
9f89fc6
Compare
lorisleiva
force-pushed
the
loris/http-transport-for-solana-rpc
branch
from
September 3, 2024 22:11
674b13b
to
6b996b2
Compare
lorisleiva
force-pushed
the
loris/is-solana-request
branch
from
September 5, 2024 09:23
9f89fc6
to
b159be0
Compare
lorisleiva
force-pushed
the
loris/http-transport-for-solana-rpc
branch
from
September 5, 2024 09:23
6b996b2
to
f6f7579
Compare
lorisleiva
force-pushed
the
loris/is-solana-request
branch
from
September 6, 2024 11:49
b159be0
to
26af60a
Compare
lorisleiva
force-pushed
the
loris/http-transport-for-solana-rpc
branch
from
September 6, 2024 11:49
f6f7579
to
e5a3363
Compare
lorisleiva
force-pushed
the
loris/is-solana-request
branch
from
September 10, 2024 07:55
26af60a
to
5c4c50b
Compare
lorisleiva
force-pushed
the
loris/http-transport-for-solana-rpc
branch
from
September 10, 2024 07:56
e5a3363
to
db92783
Compare
lorisleiva
force-pushed
the
loris/is-solana-request
branch
from
September 10, 2024 08:17
5c4c50b
to
2d06c42
Compare
lorisleiva
force-pushed
the
loris/http-transport-for-solana-rpc
branch
from
September 10, 2024 08:17
db92783
to
6163547
Compare
lorisleiva
force-pushed
the
loris/is-solana-request
branch
from
September 10, 2024 08:51
2d06c42
to
d566b79
Compare
lorisleiva
force-pushed
the
loris/http-transport-for-solana-rpc
branch
from
September 10, 2024 08:51
6163547
to
c3c60e2
Compare
lorisleiva
force-pushed
the
loris/is-solana-request
branch
from
September 10, 2024 09:01
d566b79
to
7dbb7d1
Compare
lorisleiva
force-pushed
the
loris/http-transport-for-solana-rpc
branch
2 times, most recently
from
September 10, 2024 09:07
868cda0
to
ce1d907
Compare
lorisleiva
force-pushed
the
loris/is-solana-request
branch
from
September 10, 2024 10:13
7dbb7d1
to
f2d3b70
Compare
lorisleiva
force-pushed
the
loris/http-transport-for-solana-rpc
branch
from
September 10, 2024 10:13
ce1d907
to
584fd59
Compare
steveluscher
approved these changes
Sep 10, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merge activity
|
lorisleiva
changed the base branch from
loris/is-solana-request
to
graphite-base/3196
September 11, 2024 08:22
lorisleiva
force-pushed
the
loris/http-transport-for-solana-rpc
branch
from
September 11, 2024 08:25
584fd59
to
ac099d6
Compare
Merged
Because there has been no activity on this PR for 14 days since it was merged, it has been automatically locked. Please open a new issue if it requires a follow up. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
This PR creates a new HTTP transport specific to the Solana RPC API that prevents loss of precision for large integers, in both the request and the response of the RPC cycle.
It does so by using the previously added helpers:
isSolanaRequest
: to figure out if we should customise the way we parse/stringify the response/request respectively.parseJsonWithBigInts
: for our customfromJson
option, if a Solana request has been identified.stringifyJsonWithBigInts
: for our customtoJson
option, if a Solana request has been identified.