Skip to content

Meetup script updates #6641

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

bdillo
Copy link
Contributor

@bdillo bdillo commented May 15, 2025

Changes to the meetup scripts:

  • UK events show UK as the country code in the meetup ui, but return GB. Normalize them to use UK as this seems like the more common way to write the addresses there
  • Add support for location overrides. Some meetup groups are hybrid (like Seattle and Vancouver), so we can mark them as such in our rust-meetups file. Additionally, some meetup groups don't list their location at all, so we can mark those as virtual if needed, after manually checking they are in fact virtual.
  • With the above changes, the rust-meetups file has a new format so we can add arbitrary properties for each meetup group if we want to have some specific behavior within our scripts.
  • Also remove the /events part of the meetup group URLs. The URL without the /events section lists the groups description as well as the upcoming events, whereas /events just lists the events. Just seems like a nicer URL to use.

I manually tested and validated the data, also used this script as part of the process in #6640

…pport hybrid events going in two sections, remove stale meetup groups
@bdillo bdillo requested a review from mariannegoldin May 15, 2025 00:44
"https://www.meetup.com/desert-rustaceans": {},
"https://www.meetup.com/detroitrust": {},
"https://www.meetup.com/dutch-rust-meetup": {},
"https://www.meetup.com/indyrs": {},
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"https://www.meetup.com/indyrs": {},
"https://www.meetup.com/ideas-and-coffee/": {},
"https://www.meetup.com/indyrs": {},

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