Replies: 1 comment 2 replies
-
|
Greetings @raisch thanks so much! Sorry for the delay, I have been dealing with sick family this weekend. All of that sounds great. I would be happy to just add you as a contributor if that's easier for you...? |
Beta Was this translation helpful? Give feedback.
2 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Hi.
First, thanks for SocialPredict. I'm quite excited to build my own projects using it as a base.
I have a few initial thoughts to share (below).
These are my observations when doing a cold install on a Macbook running MacOS Sequoia 15.6.1 with 16GB and 512GB disk of which 65+GB are free. I run the most recent version of Docker via Docker Desktop.
When I've fully evaluated the platform, I'll be sure to post bug reports.
And as I discover new things, I'll add them as comments on this post.
Installation
./data/postgres. This is not the case; the data files are stored in a Docker volume namedscripts_pgdatawhich, even if this was desired behavior, the name seems overly generic and so doesn't provide enough useful context../.envworks for most setup. However lines ## 62-63 in./scripts/dev/build_dev.shoverwrite the values defined in the env file with defaults, i.e.BACKEND_IMAGE_NAME="socialpredict-backend:latest"andFRONTEND_IMAGE_NAME="socialpredict-frontend:latest", which causes the Docker container creation to fail. Commenting those lines causes container creation to succeed.User Creation
Unexpected token 'N', "New passwo"... is not valid JSONwhich seems pretty odd to me.initial_account_balancein theuserstable appears to work, that value doesn't appear to surface properly in the UI, as the message: 'Error: Error: Failed to fetch user credit' appears in the left column of the UI directly below the 'SocialPredict' header.Final Thoughts: In my opinion, the new password handling logic is really very brittle and doesn't communicate error conditions to the user well. I suggest it needs significant refactoring. (If I get the time, I'll update it and submit a PR, unless someone else beats me to it.)
Thanks again for SocialPredict & all best.
Beta Was this translation helpful? Give feedback.
All reactions