-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for standalone Places, Routes SDKs #40
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
Bumps [micromatch](https://github.com/micromatch/micromatch) from 4.0.5 to 4.0.8. - [Release notes](https://github.com/micromatch/micromatch/releases) - [Changelog](https://github.com/micromatch/micromatch/blob/master/CHANGELOG.md) - [Commits](micromatch/micromatch@4.0.5...4.0.8) --- updated-dependencies: - dependency-name: micromatch dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]>
…romatch-4.0.8 Bump micromatch from 4.0.5 to 4.0.8
Bumps [fast-xml-parser](https://github.com/NaturalIntelligence/fast-xml-parser) to 4.4.1 and updates ancestor dependency [@aws-sdk/client-location](https://github.com/aws/aws-sdk-js-v3/tree/HEAD/clients/client-location). These dependencies need to be updated together. Updates `fast-xml-parser` from 4.2.5 to 4.4.1 - [Release notes](https://github.com/NaturalIntelligence/fast-xml-parser/releases) - [Changelog](https://github.com/NaturalIntelligence/fast-xml-parser/blob/master/CHANGELOG.md) - [Commits](NaturalIntelligence/fast-xml-parser@v4.2.5...v4.4.1) Updates `@aws-sdk/client-location` from 3.359.0 to 3.649.0 - [Release notes](https://github.com/aws/aws-sdk-js-v3/releases) - [Changelog](https://github.com/aws/aws-sdk-js-v3/blob/main/clients/client-location/CHANGELOG.md) - [Commits](https://github.com/aws/aws-sdk-js-v3/commits/v3.649.0/clients/client-location) --- updated-dependencies: - dependency-name: fast-xml-parser dependency-type: indirect - dependency-name: "@aws-sdk/client-location" dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]>
…ti-d875a7cb14 Bump fast-xml-parser and @aws-sdk/client-location
On further investigation, Polygon should be fully flattened, and none of the other fields listed actually appear in supported Response types.
For the new GeoPlaces APIs, each Response type is now mapped to a separate API. This makes the code easier to write and maintain, and will scale better if more APIs are added over time.
Add places V2 API support to datatypes lib
Bumps [rollup](https://github.com/rollup/rollup) from 3.22.0 to 3.29.5. - [Release notes](https://github.com/rollup/rollup/releases) - [Changelog](https://github.com/rollup/rollup/blob/master/CHANGELOG.md) - [Commits](rollup/rollup@v3.22.0...v3.29.5) --- updated-dependencies: - dependency-name: rollup dependency-type: direct:development ... Signed-off-by: dependabot[bot] <[email protected]>
MapLibre expects GeoJSON Feature ids to be integers. By placing PlaceId values in the id field, they get stripped by MapLibre into "undefined". However, because PlaceId is a valid Feature id value for uses other MapLibre, this change changes the place-converter to stop stripping PlaceId from properties and leaves the id alone for backwards compatibility. The new geoplaces-converter has been changed to put a unique integer in the id field and to stop stripping PlaceId from properties.
…lup-3.29.5 Bump rollup from 3.22.0 to 3.29.5
Fixed Places Feature id handling to be more MapLibre-friendly
This converter is more complex than the V0 converter. It exposes options for converting leg lines, travel step lines, travel step points, leg arrival/departure points, and span lines. When all of these are combined in a single GeoJSON, they can be drawn, detected, and styled differently based on the FeatureType property added to each generated Feature.
This API change replaces Query with QueryText.
Adds route converter for V2 CalculateRoutesResponse
Fixed some README method name typos
* Added V2 route converters and changed flattenProperties default to true. Added converters for CalculateIsolines, OptimizeWaypoints, and SnapToRoads. Also changed all of the flattenProperties defaults for the V2 converters to true because that's the common use case path for use with MapLibre. * Updated all comments to include examples of rendering the results. Also made everything more typedoc-friendly. * Fixed up README and comments. * Removed unneeded comments. * Addressed PR feedback. * Renamed feature types based on PR feedback. * Optimized and split the extract methods. * Fixed README to match PR feedback changes.
The UMD build should be used for browsers (and unpkg, which otherwise references `main`, which is now CommonJS).
Specify browser artifacts in package.json
The package names changed (now geo-<vertical>) and a property was removed from Routes.
* Switched to public polyline release. * Switch to released polyline package. Previous commit was relying on a locally-installed development build of @aws-geospatial/polyline, which is why it didn't cause any compile failures. * Added commonjs to the datatypes rollup
Also added a CHANGELOG.md file.
Use new public packages
Bumps [rollup](https://github.com/rollup/rollup) from 3.22.0 to 3.29.5. - [Release notes](https://github.com/rollup/rollup/releases) - [Changelog](https://github.com/rollup/rollup/blob/master/CHANGELOG.md) - [Commits](rollup/rollup@v3.22.0...v3.29.5) --- updated-dependencies: - dependency-name: rollup dependency-type: direct:development ... Signed-off-by: dependabot[bot] <[email protected]>
tjmichaels
approved these changes
Oct 30, 2024
cgalvan
approved these changes
Oct 30, 2024
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.
Description of changes:
This adds support for converting all of the new response types in the standalone Places and Routes SDKs. See the README for more details on the new methods and how to use them.