RamiAwar / dataline

Chat with your data - AI data analysis and visualization on CSV, Postgres, MySQL, Snowflake, SQLite...
https://dataline.app
GNU General Public License v3.0
502 stars 49 forks source link

Make bundling possible for easier app runs #127

Closed anthony2261 closed 3 months ago

anthony2261 commented 4 months ago

Issue #, if available:

126

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

anthony2261 commented 4 months ago

@RamiAwar can you check the BUNDLING.md file for instructions on bundling for MacOS (don't forget to run the frontend build commands at the top of the file first) and check if the bundling works well for MacOS? You should be able to run the executable called main under macos_dist/main/main at the root of the repo and get the frontend once you open localhost:7377

RamiAwar commented 4 months ago

You should be able to run the executable called main under macos_dist/main/main at the root of the repo and get the frontend once you open localhost:7377

Why 7377 not 2222 like the docker file?

anthony2261 commented 3 months ago

You should be able to run the executable called main under macos_dist/main/main at the root of the repo and get the frontend once you open localhost:7377

Why 7377 not 2222 like the docker file?

@RamiAwar Don't know why I can't directly reply to this comment but whatever, it's because that's the backend server's port. If we tell it to run on port 2222, we would need to tell the frontend to call the backend on port 2222 instead of 7377 (which is easily doable), see text2sql-frontend/src/services/api_client.ts for how we're specifying the port. Let me know we should do it on port 2222, I'll make the changes

RamiAwar commented 3 months ago

One thing still missing here is 1) conditionally running this only when we want to release and 2) creating a github release out of these artifacts. Upload artifacts are only useful for stuff needs to stay in the build. In this case we're doing a full release.

But let's do that in other PRs.