search

multiprocessio / datastation

App to easily query, script, and visualize data from every database, file, and API.
https://datastation.multiprocess.io
Other
2.9k stars 112 forks source link

Fixes #217 #218

Closed steirico closed 2 years ago

steirico commented 2 years ago

This is a fix for #217.

eatonphil commented 2 years ago

Thanks for the bug report and the suggests fix! If you'd like to make the suggested change go for it otherwise I can.

steirico commented 2 years ago

You are welcome! Feel free to go for the way you prefer. 😉

eatonphil commented 2 years ago

I opened a PR against your branch:

https://github.com/steirico/datastation/pull/1

It also adds your reproduction file as a regression test.

eatonphil commented 2 years ago

These tests are failing because of your permissions as a PR author. They are ok to fail. When the rest finish I'll merge.