Database driven organizers page

sandiegopython / pythonsd-django

http://sandiegopython.org/

MIT License

7 stars 5 forks source link

Database driven organizers page #156

Closed davidfischer closed 4 months ago

davidfischer commented 4 months ago

Adds the first migration to the app
Adds an Organizer model that will be displayed on the organizer page
This PR does not yet display images but it would be great if profile photos came from Meetup profiles.
Should these organizers have a brief description?

Replaces #155. Fixes #64.

Screenshot

Screenshot from 2024-05-30 23-03-54

Deploying

This PR adds our first migration needed for the site. As with any kind of migration, after this is merged, somebody (probably me) will need to run this migration on the production database.

Jer-Pha commented 4 months ago

Here's a CSV to easily copy/paste the names and Meetup URLs in /admin/. I don't know who is considered active so I sorted the list based on "Last visited" on Meetup.

pythonsd-organizers.csv

davidfischer commented 4 months ago

I created a separate PR (#158) that merges into this one. That PR sets up Docker Compose which I really wanted in order to test this PR with Postgres and a custom storage backend. The custom media storage backend is what I'm planning to use to store organizer photos in something like S3 (or more specifically Cloudflare R2). This PR I think is ready to review/merge and it could be merged without merging the other.

davidfischer commented 4 months ago

Looks good.

Thanks for the review. I asked in discord but I'll probably merge #158 today or tomorrow and deploy this. I don't want it to sit around too long.

Jer-Pha commented 4 months ago

I looked over the changes in #158 but don't know enough about the non-Django stuff, which is why I only mentioned it in #154. I started my Docker journey this weekend, so I'll hopefully be more helpful with future PRs.