Skip to content

fix(amplify-libraries, Android): use Kotlin instead of TypeScript in secondary index client examples - #8631

Merged
VarshithaPamisetty merged 1 commit into
mainfrom
fix/android-secondary-index-kotlin-examples
Oct 23, 2026
Merged

VarshithaPamisetty merged 1 commit into
mainfrom
fix/android-secondary-index-kotlin-examples

Conversation

@VarshithaPamisetty

@VarshithaPamisetty VarshithaPamisetty commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Description of changes:

The Android secondary index page served TypeScript client code (generateClient, client.models.*) because android was grouped into the JS InlineFilter. Replaced the three client-side sections with Kotlin examples using SimpleGraphQLRequest, since ModelQuery.list issues listCustomers(filter:) — a scan, not a GSI query. Backend amplify/data/resource.ts blocks are unchanged.

Two smaller fixes on the same page: the three added fences are tagged kotlin rather than kt, and the shared prose line above "Customize the query field for secondary indexes" no longer names the JS-only client.models.<MODEL_NAME>.listBy..., which rendered to all nine platforms.

Checks

  • yarn build succeeds with no errors

Refs #8342

@VarshithaPamisetty
VarshithaPamisetty marked this pull request as ready for review October 19, 2026 11:39
@VarshithaPamisetty
VarshithaPamisetty requested a review from a team as a code owner October 19, 2026 11:39
@VarshithaPamisetty
VarshithaPamisetty force-pushed the fix/android-secondary-index-kotlin-examples branch from 2ad74c0 to 0db2a06 Compare October 19, 2026 13:08
@VarshithaPamisetty VarshithaPamisetty self-assigned this Oct 19, 2026
@VarshithaPamisetty
VarshithaPamisetty enabled auto-merge (squash) October 19, 2026 14:28
@VarshithaPamisetty
VarshithaPamisetty merged commit b007f6f into main Oct 23, 2026
13 checks passed
@VarshithaPamisetty
VarshithaPamisetty deleted the fix/android-secondary-index-kotlin-examples branch October 23, 2026 12:25
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.

3 participants