Skip to content
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

Make the TripPin.Feed type change more clear #522

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion powerquery-docs/samples/trippin/7-advancedschema/readme.md
Original file line number Diff line number Diff line change
Expand Up @@ -188,7 +188,13 @@ The full code listing for the `Table.ChangeType` function can be found in the [T
>[!Note]
> For flexibility, the function can be used on tables, as well as lists of records (which is how tables would be represented in a JSON document).

You then need to update the connector code to change the `schema` parameter from a `table` to a `type`, and add a call to `Table.ChangeType` in `GetEntity`.
You will then need to update the `schema` parameter on `TripPin.Feed` from a `table` to a `type`.

```powerquery-m
TripPin.Feed = (url as text, optional schema as type) as table => GetAllPagesByNextLink(url, schema);
```

`GetEntity` is updated to add a call to `Table.ChangeType`.

```powerquery-m
GetEntity = (url as text, entity as text) as table =>
Expand Down