Skip to content

Refactor + Dashboard Message Streaming #7

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

Merged
merged 1 commit into from
Jun 6, 2025

Conversation

rayhanadev
Copy link
Member

This PR hits several birds with one stone :D God damn feast

Removed Commands

It removes a set of un-used and un-documented commands. Specifically:

  • admin_check_birthdays
  • admin_get_current_time
  • birthday
  • remove_birthday

It also removes the dependency on an S3 database for Wack Hacker, making development/deployment easier.

New crons/ folder

Since Wack Hacker regularly runs cron-jobs to interact with the Discord server and there are plans to add alerting (#6) it makes sense to modularize this functionality and make adding more cron-jobs easier.

Restructure events/

This folder was arranged in a really crappy way. I made the new standard events/<event_name>/<action-type>.ts instead of having all the events in one event file.

Dashboard Message Streaming

See purduehackers/api-v3#1 this PR adds the Discord Bot side support for streaming live Discord messages to the dashboard.

@rayhanadev rayhanadev self-assigned this Jun 6, 2025
@rayhanadev rayhanadev merged commit 043b16d into main Jun 6, 2025
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.

1 participant