Skip to content

Commit

Permalink
Clarify possible trade_type filter values in README. (stellar#706)
Browse files Browse the repository at this point in the history
  • Loading branch information
Shaptic authored Sep 10, 2021
1 parent bb58357 commit 45bb962
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/npm_publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,9 @@ jobs:
run: yarn

- name: Build
run: gulp
run: yarn version

- name: Publish npm package
run: npm unpublish stellar-sdk@v9.0.0 && yarn publish --tag beta
run: yarn publish --tag beta
env:
NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
2 changes: 1 addition & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ This beta release adds **support for Automated Market Makers**. For details, you
* `/liquidity_pools/:id/effects`

- Expanded the `TradesCallBuilder` to support fetching liquidity pool trades and accepts a new `trade_type` filter ([#685](https://github.com/stellar/js-stellar-sdk/pull/685)):
* `/trades?trade_type={orderbook,liquidity_pools}`
* `/trades?trade_type={orderbook,liquidity_pools,all}`. By default, the filter is `all`, including both liquidity pool and orderbook records.
* A liquidity pool trade contains the following fields:
- `liquidity_pool_fee_bp`: LP fee expressed in basis points, and *either*
- `base_liquidity_pool_id` or `counter_liquidity_pool_id`
Expand Down
6 changes: 3 additions & 3 deletions test/integration/apiary.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
// https://docs.google.com/document/d/1pXL8kr1a2vfYSap9T67R-g72B_WWbaE1YsLMa04OgoU/edit
const _ = require("lodash");

const MOCK_SERVER = "http://private-anon-a06e1b25a0-ammmock.apiary-mock.com";
const MOCK_SERVER = "https://private-d133c-ammmock.apiary-mock.com";

describe("tests the /liquidity_pools endpoint", function() {
const lpId = "0569b19c75d7ecadce50501fffad6fe8ba4652455df9e1cc96dc408141124dd5";
Expand Down Expand Up @@ -55,7 +55,7 @@ describe("tests the /liquidity_pools endpoint", function() {
};

Object.keys(testCases).forEach(suffix => {
it(`GET /<id>/${suffix}`, function(done) {
it(`GET /<pool-id>/${suffix}`, function(done) {
chai.request(MOCK_SERVER)
.get(`/liquidity_pools/${lpId}/${suffix}`)
.end(function(err, res) {
Expand Down Expand Up @@ -129,7 +129,7 @@ describe("tests the /accounts endpoint", function() {
});
});

it('GET /<id>', function(done) {
it('GET /<account-id>', function(done) {
const accountId = "GDQNY3PBOJOKYZSRMK2S7LHHGWZIUISD4QORETLMXEWXBI7KFZZMKTL3";

chai.request(MOCK_SERVER)
Expand Down

0 comments on commit 45bb962

Please sign in to comment.