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

Meetup json output, minor fixes #6339

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

bdillo
Copy link
Contributor

@bdillo bdillo commented Feb 2, 2025

This fixes some of the event location capitalization issues. The meetup API returns strings like "San francisco", so this fix will format these things a bit more correctly. Also in GB part of the postal code was used as the state, this is ignored now. Additionally, the meetup API returns an empty string if a part of a location is missing, which we now switch to None.

This also adds a way to output the events as json so they can be used in further automation. Currently we're just merging the lists together manually, having a better output format to use will let us create other tools to merge the lists together and save us some time.

Change was tested manually, example json partial output:

{"Virtual": [{"name": "Indy.rs - with Social Distancing", "location": "Indianapolis, IN, US", "date": "2025-02-05", "url": "https://www.meetup.com/indyrs/events/302031658", "virtual": true, "organizer_name": "Indy Rust", "organizer_url": "https://www.meetup.com/indyrs/events/"}, ...

@bdillo bdillo requested a review from mariannegoldin February 2, 2025 01:46
@bdillo bdillo self-assigned this Feb 2, 2025
@bennyvasquez
Copy link
Contributor

I'm going through PRs in prep for this week's edition. I'm not at all qualified to review this one (which is why you didn't ask me to!) but I'm excited for it nonetheless! We'll leave it for @mariannegoldin. :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants