chore: Update v2 docs #239
Conversation
samtin0x
force-pushed
the
samtin0x/update-v2-docs
branch
from
6696d08
to
d6fac07
Compare
| rest_indexer=REST_URL, | ||
| websocket_indexer=WEBSOCKET_URL | ||
| node_url="dydx-ops-grpc.kingnodes.com:443", # No 'https://' prefix | ||
| rest_indexer="https://indexer.v4.dydx.exchange", |
There was a problem hiding this comment.
@Christopher-Li are we okay with suggesting indexer endpoints via docs? if so this looks like a pretty solid upgrade from our current documentation
There was a problem hiding this comment.
@samtin0x i'd prefer if we keep these listed as variable names and instead link to https://docs.dydx.exchange/infrastructure_providers-network/resources#indexer-endpoints.
This will make the docs more durable to changes in indexer endpoints and also let developers know that they can set up their own indexer or node urls
There was a problem hiding this comment.
Yes, both approaches work. As long as users/devs have a simple example they can copy paste and get quickstarted, as that has been the main blocker here, hence the PR
samtin0x
force-pushed
the
samtin0x/update-v2-docs
branch
2 times, most recently
from
eb2de48
to
3bfd0f8
Compare
| rest_indexer=REST_URL, | ||
| websocket_indexer=WEBSOCKET_URL | ||
| node_url="NODE_URL", # No 'https://' prefix | ||
| rest_indexer="REST_INDEXER", |
There was a problem hiding this comment.
[nit] should still be descriptive of the data type REST_INDEXER_URL
samtin0x
force-pushed
the
samtin0x/update-v2-docs
branch
from
3bfd0f8
to
50cb3f1
Compare
No description provided.