Skip to content

Commit

Permalink
Merge pull request #268 from Element84/pv/bidx
Browse files Browse the repository at this point in the history
pass mosaicking bidx param as multiple params instead of csv
  • Loading branch information
Phil Varner authored Oct 3, 2023
2 parents 1527c71 + 25fc51e commit 46db98d
Show file tree
Hide file tree
Showing 4 changed files with 21 additions and 8 deletions.
9 changes: 8 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,14 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).

## Unreleased
## 4.2.0 - 2023-10-03

### Changed

- Mosaic tiling parameter `bidx` is now passed as multiple parameters
(e.g., `bidx=1&bidx=2&bidx=3`) instead of a single comma-delimited value
(e.g., `bidx=1,2,3`). This is to accomodate a change to titler-mosaicjson
as of v0.14.0

## 4.1.0 - 2023-09-14

Expand Down
13 changes: 8 additions & 5 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "filmdrop-ui",
"version": "4.1.0",
"version": "4.2.0",
"license": "Apache-2.0",
"dependencies": {
"@emotion/react": "^11.11.1",
Expand Down
5 changes: 4 additions & 1 deletion src/utils/mapHelper.js
Original file line number Diff line number Diff line change
Expand Up @@ -485,7 +485,10 @@ const parameters = {
const assetBidx = asset && value ? `${asset}|${value}` : null
return assetBidx && `asset_bidx=${assetBidx}`
} else {
return value && `bidx=${value}`
return value
?.split(',')
.map((x) => `bidx=${x}`)
.join('&')
}
}
}
Expand Down

0 comments on commit 46db98d

Please sign in to comment.