-
Notifications
You must be signed in to change notification settings - Fork 1.3k
[Maps] Onboarding @azure-rest/maps-route
#23588
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
Merged
andykao1213
merged 361 commits into
Azure:main
from
andykao1213:maps/onboarding-maps-route-RLC
Oct 26, 2022
Merged
Changes from 250 commits
Commits
Show all changes
361 commits
Select commit
Hold shift + click to select a range
b66237d
Update maps dependency in pnpm-lock.yaml
64de5e9
Fix sample metadata errors
8248dea
Fix linting errors
75cf90f
Fix linting errors
dubiety 7863b9e
Fix run time error
ff5d8a5
Update package.json in samples folder
cce0f9e
Fix linting errors
dubiety 45eae6e
Fix linting errors
dubiety 33d39a7
Add changelog for maps-common
fe0ede3
Update recording files for maps-search
0f2d40c
Add test.yml for maps-search
e8776d5
Add tests.yml for render and route
098ccdc
Update tests.yml
3a4f796
Update test-resources.json for maps
dubiety c529bdd
Fix tests.yml format error
dubiety 70596eb
Update env vars and remove unused export var
dc04541
Remove unused code
dubiety 1365891
Fix lock file
andykao1213 9cea83b
Remove route & render
andykao1213 fd5ac05
Replace `beginGet*Batch` with `resume*Batch`
andykao1213 ea49a98
Update recording
andykao1213 688a2e0
Make the endpoint of `MapsSearchClient` configurable
andykao1213 c18836e
Change LatLon from an interface to a tuple
andykao1213 262e74c
Make the props `coordinates` and `countryCodeFilter` optional
andykao1213 64ab74f
fixup! Remove route & render
andykao1213 18f84ec
Adjust CHANGELOGs of maps-search & common
andykao1213 ea70218
Update samples
andykao1213 b888756
Add batchRequest sample
andykao1213 c03e5b4
Add a new sample searchAddressResult
andykao1213 f86afb8
Update search.ts and its examples
andykao1213 de2378c
enhance code example in README.md
andykao1213 4a922cc
comment unused import
andykao1213 e4780e7
Update recording for maps-search
andykao1213 bf4bd29
remove redundant recording files
andykao1213 9ca9ff2
Add maps-search
alankashiwa 84e0f98
Rename LatLong to LatLon
alankashiwa b4fadd6
Fix constructor Readme doc
alankashiwa 5bb423c
Update readme to new signature
alankashiwa e1397b0
Add tests partially
alankashiwa 7aafe23
Change LatLon/BoundingBox to interfaces
alankashiwa 55bec4e
Reduce the number of constructor overloading
alankashiwa d9b6413
Rename SearchClient as MapsSearchClient
alankashiwa 82aa02d
Add options mapper tests
alankashiwa 979d879
Add result type mapper tests (in progress)
alankashiwa 360697d
rename clientId to mapsAccountClientId
alankashiwa 67b123d
listPolygons -> getPolygons
alankashiwa 2a6f55c
getPointOfInterestCategoryTree -> getPointOfInterestCategories
alankashiwa 077981c
Expose missing model
alankashiwa de87099
Add alternative input file for quick experiment
alankashiwa 01aa61f
Regenerate code and make changes accordingly
alankashiwa 8016547
Reduce number of overloads of fuzzy searches
alankashiwa 7d19057
Add geojson types
alankashiwa 2cdc2b5
Adopting new geojson definitions
alankashiwa 752328d
Change test specs to align with new method names
alankashiwa 7f86f4c
clean up models and tsconfig
alankashiwa fedd708
Expose Known* models
alankashiwa 5136fbd
Remove batch sync methods
alankashiwa 12c08fa
Add public tests (non-batch)
alankashiwa 9367451
move model
alankashiwa b660b01
Add docstrings
alankashiwa 610eb89
Start polling in begin* method
alankashiwa 975416f
Refactor BatchPoller for retrieving batchId
alankashiwa abd2489
Finalize batch pollers
alankashiwa e2698d3
Readd batch methods
alankashiwa 3328be7
Update generated code
alankashiwa 72a1af8
Finalize batch signature
alankashiwa 878ae76
Rebuild samples & clean up
alankashiwa 8b55ebd
Add AAD tests
alankashiwa f7484ce
Clean up Swagger readme
alankashiwa 16a799a
Fix spelling
alankashiwa 14c6ad2
Add initial files for @azure/maps-route module
alankashiwa f07934a
maps-search: change comments and perform formatting
alankashiwa 808b008
Methods options and testing skeleton for maps-route
alankashiwa 6d57aa5
maps-search: refactor options
alankashiwa 7f60e28
Improve convenience layer for non-LRO methods
alankashiwa 0884373
Change variable names
alankashiwa d04a87d
Complete batch methods
alankashiwa 0a3f333
update maps-search Readme
alankashiwa 88a6b84
Update maps-route Readme
alankashiwa 6283712
Add generated samples for maps-route
alankashiwa d80e711
Update samples for maps-search
alankashiwa 1e468c7
Delete backup file
alankashiwa 9832fae
Add maps-common module
alankashiwa ebb11bc
Update maps-route/maps-search to use maps-common
alankashiwa 9a0a757
maps-render: first draft
alankashiwa 00b67e5
Use LatLon/BoundingBox from maps-common
alankashiwa 6c21c75
Complete maps-render with tests
alankashiwa 2ae0c92
Expose required types from maps-route
alankashiwa 2a3c091
Expose required types from maps-search
alankashiwa 84043e4
Complete unit tests for maps-render
alankashiwa 59790d5
Update maps-render README
alankashiwa 211bf59
Update samples
alankashiwa aa1f63c
Fix comment/strings
alankashiwa 4886901
Fix comment/strings
alankashiwa d27c798
Update Route comment/interface
alankashiwa 2e8be82
Regenerate code
alankashiwa 5ee3714
Update files
alankashiwa 8f5ea4f
Define required fields and fix code
alankashiwa 9f25889
Remove non-poller batch methods
alankashiwa d12d28b
Fix playback test failure
alankashiwa a47663c
Rename parts of model names
alankashiwa d3f5a7c
Rename remaining parts of models according to review feedback
alankashiwa ba8dd58
Remove TODO comments
alankashiwa de4578d
Update Reamdme transform to reflect latest swagger
alankashiwa 0ff85a9
Change key policy name
alankashiwa 161eea7
Simplify batch item error model
alankashiwa 24fc0a7
Add brower tests
alankashiwa 04450d3
Turn maps-common into internal module
alankashiwa 99fac9e
Rearrange Readme
alankashiwa ca5ebb3
Rewrite Readme for common
alankashiwa acd2d18
Remove batch-sync methods
alankashiwa dc8130e
Route required fields: part 1
alankashiwa 0484dd5
Route required fields: part 2
alankashiwa 227e278
Add AAD back
alankashiwa e1039a9
Route model fixes #1
alankashiwa deb143a
Route model fixes #2 + geojson named tuples
alankashiwa 51c0361
Refine geojson types
alankashiwa 6747930
Aliased a few method options
alankashiwa 5cd478d
maps-render required fields part 1
alankashiwa ad80121
Do not validate budget parameters
alankashiwa 3209919
Revert "Do not validate budget parameters"
alankashiwa 4c2cc89
Rename MapTileset to TileJson
alankashiwa ced730e
Render: model rename & type improvement
alankashiwa b06d6c2
Explicitly transform image range
alankashiwa 1c1a5ed
Fix route record tests
alankashiwa 3e58aa8
Update map account creation commands in Readme
alankashiwa 8fa8c35
Update Readme/docstrings/samples for Search/Route/Render
alankashiwa 15f3ea0
Remove rollup and use dev-tool instead
dubiety a60e154
Remove some empty tests
c284c7c
Add AAD test case and make file name shorter
df31dbb
Update test cases
856fbb3
Update cspell rules
dubiety 49cea8e
Update recording files
dubiety b417a0e
Simplify test cases
dubiety 030e416
Sync cspell.json from upstream
fe95b21
Use test-recorder 2.0 and update unit test cases
2bb38fb
Push wrong browsers recording files
d84d7fc
Update settings to support recording
dubiety 959fad3
Fix dependency errors
dubiety 1ed66fb
Upgrade api-recorder and fix errors
1c969b1
Code refinement
7a689f7
Update maps search recording files
d310f1f
Update karma.conf.js format
7b3de33
Update recording files and settings
ea8666d
Fix broken links
7693990
Update maps dependency in pnpm-lock.yaml
64397ef
Fix sample metadata errors
79f4503
Fix linting errors
1198269
Fix linting errors
dubiety 48fd646
Fix run time error
3b839ed
Update package.json in samples folder
1432a51
Fix linting errors
dubiety 56a6ccd
Fix linting errors
dubiety 859d199
Add changelog for maps-common
27e0ee1
Update recording files for maps-search
478b1e2
Add test.yml for maps-search
b8c88d9
Add tests.yml for render and route
45079a5
Update tests.yml
218dd52
Update test-resources.json for maps
dubiety ecb26b4
Fix tests.yml format error
dubiety 1efc226
Update env vars and remove unused export var
96d8590
Remove unused code
dubiety e29b20d
Fix lock file
andykao1213 51506f3
Remove maps-render
andykao1213 26d2878
Merge branch 'andrewkao/maps-search' into andrewkao/maps-route
andykao1213 d48d5ac
fixup! Change LatLon from an interface to a tuple
andykao1213 5b8b57e
Comment the future doc link
andykao1213 435c06a
Update doc & fix type issue
andykao1213 bce35b2
Merge remote-tracking branch 'upstream/main' into andrewkao/maps-search
andykao1213 7d06868
fix lint issues
andykao1213 7cc9e9e
Merge remote-tracking branch 'upstream/main' into andrewkao/maps-search
andykao1213 467ce32
Regenerate API View
andykao1213 0e926dd
Comment the whole [apiref] since the original syntax cannot pass the …
andykao1213 6927922
Fix issue related to the change of maps-common
andykao1213 2be3c3b
Update test recording
andykao1213 8f3b125
Remove maps-common from the Artifacts
andykao1213 58afa84
remove unused dependencies
andykao1213 19da7ce
Add impression in README so we can track the page views
andykao1213 f28c0f3
Remove `AZURE_LOG_LEVEL` in test.yml
andykao1213 f105d20
Update samples
andykao1213 da93ad9
Patch {} to properties for GeoJsonPolygon
andykao1213 d9618a7
rename maxDetourTime to maxDetourTimeInSeconds
andykao1213 70e7ddd
Update recordings
andykao1213 1b1508c
Remove any
andykao1213 b5390e3
Rename to detourTimeInSeconds to align the convention
andykao1213 1fd4459
Change the test.yml EnvVar
andykao1213 a620930
Merge remote-tracking branch 'upstream/main' into andrewkao/maps-search
andykao1213 5d7cf7d
Merge remote-tracking branch 'upstream/main' into andrewkao/maps-search
andykao1213 d28da36
Rename numResults to numberResults
andykao1213 979cae7
Add document comment
andykao1213 502d7dc
Merge remote-tracking branch 'upstream/main' into andrewkao/maps-search
andykao1213 379d411
Merge branch 'andrewkao/maps-search' into andrewkao/maps-route
andykao1213 0086c90
Add endpoint options to MapsRouteClient
andykao1213 0ba4cbc
Remove unused dependencies in maps-route
andykao1213 3faaefb
fixup! Change the test.yml EnvVar
andykao1213 862e3d8
Update interfaces & documents
andykao1213 03119aa
Update changelog
andykao1213 d594729
Merge getRouteDirection with the additional parameter one
andykao1213 34a4bff
Update samples
andykao1213 9100bda
Make the `maps-common to a published package
andykao1213 76d9057
Update the import of `maps-common` in `maps-search`
andykao1213 a7e33ec
Merge remote-tracking branch 'upstream/main' into andrewkao/maps-search
andykao1213 02a0517
Fix CI Issue
andykao1213 fd7722f
Add documents for geojson
andykao1213 5492bb7
Update the samples & readme
andykao1213 2b8a317
Merge branch 'andrewkao/maps-search' into andrewkao/maps-route
andykao1213 fad6693
Leverage maps-common package in mpas-route
andykao1213 384217c
chore update
andykao1213 710745d
Remove unnecessary eslint config
andykao1213 01f4e60
Merge remote-tracking branch 'upstream/main' into andrewkao/maps-search
andykao1213 e4258c3
Merge branch 'andrewkao/maps-search' into andrewkao/maps-route
andykao1213 62e9431
Merge remote-tracking branch 'upstream/main' into andrewkao/maps-route
andykao1213 bb920cc
WIP: Remove LRO generation
andykao1213 06cc76d
WIP: refactor mapsRouteClient
andykao1213 dd53c4d
WIP: first generation
andykao1213 3e85fe2
Format the generated code
andykao1213 93daaf5
Support AAD
andykao1213 212fc30
Transform the model name
andykao1213 1650b8d
Add helper functions
andykao1213 d17ee38
Add a sample - route.ts
andykao1213 8269886
Add samples for lro
andykao1213 ccd165b
Add test
andykao1213 30bffae
Generate samples
andykao1213 6e4cdf3
Add api view
andykao1213 1be8286
Update the generation
andykao1213 e2805dc
WIP: ReadMe
andykao1213 04b3765
Rename "computeBestOrder" to "computeBestWaypointOrder"
andykao1213 4a806dd
Refine README
andykao1213 70fc2aa
Rename model
andykao1213 f3030f2
Add test createRouteDirectionsBatchRequest
andykao1213 1bb753e
Fix lint issue
andykao1213 dd1593b
Add missing metadata
andykao1213 066b9bd
Remove maps-route/
andykao1213 96781dc
Update CI issue
andykao1213 f263b5e
Fix merged issue
andykao1213 ff4a0e1
Merge remote-tracking branch 'upstream/main' into maps/onboarding-map…
andykao1213 ff70bf4
Update version rule for maps-common
andykao1213 a31c816
Update CI name
andykao1213 912873d
Remove extra lint config
andykao1213 cd41516
Use `require` in the README for consistency
andykao1213 f389835
Regenerate the samples
andykao1213 1735ec6
Remove redundant recording files
andykao1213 d0d050d
Remove unused parameters in test-resources.json
andykao1213 39d891f
Refine readme snippet
andykao1213 20a52a2
Log something specific for the result
andykao1213 5e403e0
Update samples
andykao1213 8154f23
Split the route.ts into 3
andykao1213 0e52644
Remove model rename
andykao1213 eecbf53
Use default export for RLC factory function
andykao1213 3e039bc
Merge remote-tracking branch 'upstream/main' into maps/onboarding-map…
andykao1213 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
Oops, something went wrong.
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.
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.
@ckairen I wonder if there is a way to ignore all folders suffixed with
-restfrom cspell. Rest clients don't do any renames on the service properties and names and are autogenerated.