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

Add daily reminders for meetups happening today #3

Closed
wants to merge 4 commits into from

Conversation

ebanner
Copy link
Contributor

@ebanner ebanner commented Jul 26, 2024

This PR adds the capability to post daily reminders for meetup events happening today to slack:

Daily reminders

image

Fixes #1

Extra

It also adds the capability to replace the houly 60 minute event reminders in the Tampa Devs slack that are currently going to #meetup-reminders to address #4 (though actually replacing those are outside of the scope of this PR), and makes some small tweaks to the weekly event reminders (like posting events happening just this week instead of two weeks out).

The main.rb script would now be invoked as ruby main.rb [--daily|--hourly|--weekly]. By default ruby main.rb with no arguments defaults to main.rb --weekly as to maintain backwards compatibility with how the script is currently being invoked.

@ebanner
Copy link
Contributor Author

ebanner commented Jul 26, 2024

cc @chtzvt I am curious for your feedback on this 👀

@ebanner ebanner force-pushed the add-daily-reminders branch 5 times, most recently from 5f69382 to 0e89ba2 Compare September 12, 2024 15:18
@ebanner ebanner changed the title Add hourly and 60 minute meetup/event reminders Add daily meetup reminders Sep 12, 2024
@ebanner ebanner changed the title Add daily meetup reminders Add reminders for meetups happening today Sep 12, 2024
@ebanner
Copy link
Contributor Author

ebanner commented Sep 12, 2024

Here are what the hourly reminders would look like:

image

And weekly reminders:

image

There are some minor styling changes to weekly announcements (changing the header text to a header type). I also changed it to show events just this week, as opposed to the next two weeks, and changed the header text to reflect that change.

@ebanner
Copy link
Contributor Author

ebanner commented Sep 12, 2024

@chtzvt I'd really like to make this feature happen 🙂

@ebanner
Copy link
Contributor Author

ebanner commented Sep 12, 2024

@iBotPeaches might you take a look?

@ebanner ebanner force-pushed the add-daily-reminders branch 3 times, most recently from 26ba138 to 2ad992f Compare September 15, 2024 13:56
@ebanner ebanner marked this pull request as draft September 15, 2024 13:56
@ebanner
Copy link
Contributor Author

ebanner commented Sep 15, 2024

After chatting a bit with Charlton yesterday, I'm going to add a --destinations argument to specify which slack groups we want the daily reminders to go (basically just to TD or to TD+TBT+TBUX). Also, I'll add a new GitHub Actions workflow to run on a daily cadence.

@ebanner ebanner changed the title Add reminders for meetups happening today Add daily reminders for meetups happening today Sep 15, 2024
@ebanner ebanner force-pushed the add-daily-reminders branch 4 times, most recently from c09d6f1 to 2c327bc Compare September 17, 2024 16:18
@ebanner
Copy link
Contributor Author

ebanner commented Sep 17, 2024

Couldn't figure out how to rebase the commit cleanly, so I'm moving this PR over to #7

@ebanner ebanner closed this Sep 17, 2024
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.

Post daily meetup event reminders
1 participant