Update description of 'featuredProperties' to be more comprehensive. #2014
Conversation
jonathan-codaio
left a comment
jonathan-codaio
left a comment
There was a problem hiding this comment.
Thanks for this!
You'll need to generate the downstream documentation files by running make build and then committing those. As a one-time setup step (and then periodically if dependencies change) you'll need to run make bootstrap first.
But, we've never had people outside of Coda try this so you might encounter some errors in those commands. If you do, just shout back and we can build and commit the generated files for you.
ekoleda-codaio
left a comment
ekoleda-codaio
left a comment
There was a problem hiding this comment.
Thanks for the contribution!
schema.ts
Outdated
| * columns for any other properties, as desired. | ||
| * columns for any other properties, as desired. All featured properties need to be top-level. | ||
| * If you can't or don't want to change the received data format, consider changing the | ||
| * received object after fetching and before returning and assigning it to the scheme. |
There was a problem hiding this comment.
"scheme" => "schema".
|
Whether because this is a fork or because it's a third-party author, it looks like our continuous integration verification isn't running on this PR, plus it needs a |
|
Awesome! Thanks! |
3 participants
As discussed with @ekoleda-codaio in the forum, there is some misunderstanding about how featuredProperties can and can't be used and the workarounds existing for these cases. This additional description should clear up some of that - it would've for me 👍