Skip to content

Feature/8972 task add das tiler point lookup and batched multi collection products endpoin - #321

Merged
utas-raymondng merged 4 commits into
mainfrom
feature/8972-task-add-das-tiler-point-lookup-and-batched-multi-collection-products-endpoin
Aug 12, 2026
Merged

Feature/8972 task add das tiler point lookup and batched multi collection products endpoin#321
utas-raymondng merged 4 commits into
mainfrom
feature/8972-task-add-das-tiler-point-lookup-and-batched-multi-collection-products-endpoin

Conversation

@leslieduan

@leslieduan leslieduan commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Add DAS tiler point lookup endpoint

@utas-raymondng utas-raymondng left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor comment

String encodedVariable = URLEncoder.encode(variablePart, StandardCharsets.UTF_8);

if (canVisual) {
entry.put("visual_tile_url_template",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use UriComponentsBuilder is easier to read

@utas-raymondng utas-raymondng Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please do not duplicate function call, try to generalized it if you need multiple products return

…re/8972-task-add-das-tiler-point-lookup-and-batched-multi-collection-products-endpoin

@utas-raymondng utas-raymondng left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@utas-raymondng
utas-raymondng merged commit ecd78f4 into main Aug 12, 2026
4 checks passed
@utas-raymondng
utas-raymondng deleted the feature/8972-task-add-das-tiler-point-lookup-and-batched-multi-collection-products-endpoin branch August 12, 2026 05:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants