close
Skip to content

Implement table widget for Ecto queries#34

Merged
jonatanklosko merged 5 commits intomainfrom
jk-ecto
Jul 23, 2021
Merged

Implement table widget for Ecto queries#34
jonatanklosko merged 5 commits intomainfrom
jk-ecto

Conversation

@jonatanklosko
Copy link
Copy Markdown
Member

Mostly useful with Kino as a dev dependency in Mix projects, but may also be useful for Ecto tutorial notebooks.

kino_ecto.mp4

@jonatanklosko
Copy link
Copy Markdown
Member Author

Here is a self-contained example notebook.

Comment thread lib/kino/ecto.ex Outdated
Comment thread lib/kino/utils/table.ex Outdated
Comment thread lib/kino/utils/table.ex Outdated
Comment thread lib/kino/utils/table.ex
Comment thread lib/kino/ecto.ex
Comment thread lib/kino/ecto.ex
Copy link
Copy Markdown
Contributor

@josevalim josevalim left a comment

Choose a reason for hiding this comment

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

Comments added. We should probably devise a testing scheme where we emulate the messages to handle some of the corner cases.

Comment thread lib/kino/utils/table.ex Outdated
Comment thread lib/kino/utils/table.ex
jonatanklosko and others added 2 commits July 23, 2021 10:49
Co-authored-by: José Valim <jose.valim@dashbit.co>
@jonatanklosko jonatanklosko merged commit a3092e6 into main Jul 23, 2021
@jonatanklosko jonatanklosko deleted the jk-ecto branch July 23, 2021 11:34
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