Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Database works, UI got an upgrade, authentication not done yet #1

Merged
merged 25 commits into from
Sep 24, 2024

Conversation

ArondenOuden
Copy link
Contributor

Made the existing project work locally.
Project now uses postgresql using psycopg2 in python which wasn't properly set up.
Now it should work if you install postgrsql on your machine and then you can view/edit your current database using PGAdmin 4.

Made the UI theme purple and added custom markers. Filled the pop-ups with content I found could be fun. I added the basics for 'spots' which are the amount of times a sticker has been spotted. It could be expanded by providing a photo (and maybe verifying each spot), but I now works with a simple button.

Added a 'Google Maps style' 'near you' content overlay thing. Still a bit of a work in progress but the general idea is working well.

The database connection works, but the authentication currently doesn't. I also (temporarily) made the stickers verified by default, as the admin page isn't working at the moment.

@SilasPeters SilasPeters added the enhancement New feature or request label May 6, 2024
@SilasPeters SilasPeters self-requested a review May 6, 2024 18:38
@SilasPeters SilasPeters force-pushed the BasicsWorking branch 3 times, most recently from f94973d to 621d4bb Compare May 6, 2024 19:21
Copy link
Member

@SilasPeters SilasPeters left a comment

Choose a reason for hiding this comment

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

My psql installation is broken on my system, so I can't test it as of now. Will continue fixing it later, but here's my review so far.

static/favicon.ico Outdated Show resolved Hide resolved
sample.env Outdated Show resolved Hide resolved
static/js/main.js Show resolved Hide resolved
static/uploads/972CA94D-8DD5-41EC-B7C4-4B2CA8C1BE30.jpeg Outdated Show resolved Hide resolved
server.py Show resolved Hide resolved
server.py Show resolved Hide resolved
server.py Show resolved Hide resolved
server.py Show resolved Hide resolved
@SilasPeters
Copy link
Member

SilasPeters commented May 30, 2024

I will re-review this soon. Please refrain from pushing new features in this branch, for otherwise I will get in a endless review cycle. Although, your latest push was done by me haha

Copy link
Member

@SilasPeters SilasPeters left a comment

Choose a reason for hiding this comment

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

Feedback so far. Good job though, you're working hard!

static/js/main.js Outdated Show resolved Hide resolved
.gitignore Show resolved Hide resolved
Copy link
Member

@SilasPeters SilasPeters left a comment

Choose a reason for hiding this comment

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

Error upon boot, something with static/uploads/ being ignored.

Our idea was to let server.py craete the folder on boot, if it does not exist yet

Copy link
Member

@SilasPeters SilasPeters left a comment

Choose a reason for hiding this comment

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

LGTM!

@SilasPeters SilasPeters merged commit c00c95d into svsticky:master Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants