Skip to content

remove: legacy events system#236

Merged
danielhe4rt merged 4 commits into
4.xfrom
remove/legacy-events
May 16, 2026
Merged

remove: legacy events system#236
danielhe4rt merged 4 commits into
4.xfrom
remove/legacy-events

Conversation

@danielhe4rt
Copy link
Copy Markdown
Contributor

@danielhe4rt danielhe4rt commented May 16, 2026

Summary

  • Remove the legacy events module (models, enums, actions, migrations, and pivot tables)
  • Add a migration to drop all legacy event-related tables (events, events_talks, events_attendees, sponsors, events_sponsors, events_agenda, event_submission_speakers)
  • Clean up references to the old events system that was replaced by the new timeline/social feed

Context

The events module was superseded by the new timeline social feed (#223) and moderation system. The old models (EventModel, EventAgenda, EventSegment, EventSubmission, etc.) and their associated enums, actions, and pivot tables are no longer used.

Test plan

  • Run php artisan migrate to confirm the drop migration executes cleanly
  • Run php artisan test --compact to verify no remaining references to removed classes
  • Confirm no Filament resources reference the deleted models

Description

This PR removes the legacy events system (models, enums, actions, and related database structures) that was superseded by the new timeline social feed (#223) and moderation system. The changes include deletion of 26 files—database migrations, Eloquent models, pivot tables, enums, and action classes—replaced by a single migration that drops all event-related tables in a single transaction.

References

Contributor Summary

Contributor Lines Added Lines Removed Files Changed
danielhe4rt 20 1,271 27

Changes Summary

File Path Change Description
app-modules/events/database/migrations/2025_11_05_191403_create_events_table.php Removed migration creating events table
app-modules/events/database/migrations/2025_11_05_192008_create_events_talks_table.php Removed migration creating events_talks table
app-modules/events/database/migrations/2025_11_05_192756_create_events_attendees_table.php Removed migration creating events_attendees pivot table
app-modules/events/database/migrations/2025_11_05_193042_create_sponsors_table.php Removed migration creating sponsors table
app-modules/events/database/migrations/2025_11_05_193141_create_events_sponsors_table.php Removed migration creating events_sponsors pivot table
app-modules/events/database/migrations/2025_11_27_132714_add_attend_order_to_events_attendees_table.php Removed migration adding attend_order column
app-modules/events/database/migrations/2025_11_27_145728_create_events_agenda_table.php Removed migration creating events_agenda table
app-modules/events/database/migrations/2025_11_27_153537_remove_starts_at_ends_at_columns_to_events_talks_table.php Removed migration removing timestamp columns
app-modules/events/database/migrations/2025_11_27_171411_create_event_submission_speakers_table.php Removed migration creating event_submission_speakers table
app-modules/events/database/migrations/2026_05_16_195205_drop_events_module_tables.php Added new migration dropping all legacy event tables
app-modules/events/src/Actions/AttendEventAction.php Removed action class for event attendance
app-modules/events/src/Actions/LeaveEventAction.php Removed action class for leaving events
app-modules/events/src/Enums/AttendingStatusEnum.php Removed enum for attendance statuses
app-modules/events/src/Enums/EventTypeEnum.php Removed enum for event types
app-modules/events/src/Enums/SchedulableTypeEnum.php Removed enum for schedulable types
app-modules/events/src/Enums/SponsoringLevelEnum.php Removed enum for sponsoring levels
app-modules/events/src/Enums/Talks/TalkStatusEnum.php Removed enum for talk statuses
app-modules/events/src/Models/EventAgenda.php Removed EventAgenda model
app-modules/events/src/Models/EventModel.php Removed main EventModel with relationships and operations
app-modules/events/src/Models/EventSegment.php Removed EventSegment model
app-modules/events/src/Models/EventSubmission.php Removed EventSubmission model
app-modules/events/src/Models/Pivot/EventAttend.php Removed EventAttend pivot model
app-modules/events/src/Models/Pivot/EventSubmissionSpeaker.php Removed EventSubmissionSpeaker pivot model
app-modules/events/src/Models/Pivot/SponsorAttend.php Removed SponsorAttend pivot model
app-modules/events/src/Models/Sponsor.php Removed Sponsor model

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 16, 2026

Warning

Rate limit exceeded

@danielhe4rt has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 26 minutes and 58 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Pro

Run ID: 9bd29bab-a29c-4fe0-9e31-6719f61eaa71

📥 Commits

Reviewing files that changed from the base of the PR and between 52a0bb9 and 036aebc.

📒 Files selected for processing (8)
  • app-modules/events/src/EventsServiceProvider.php
  • app-modules/identity/src/Tenant/Models/Tenant.php
  • app-modules/identity/src/User/Models/User.php
  • app-modules/integration-discord/src/ETL/Actions/MergeDuplicateDiscordUserAction.php
  • app-modules/panel-admin/tests/Feature/ApplyTenantScopesTest.php
  • app/Rules/AvailableTalkSchedule.php
  • database/migrations/2026_03_21_192333_convert_morph_types_to_aliases.php
  • database/seeders/BaseSeeder.php
📝 Walkthrough

Walkthrough

This PR removes the entire events module from the codebase and database. Seven existing migrations that create events-related tables are deleted, along with model classes (EventModel, EventAgenda, EventSubmission, Sponsor, and related pivot models), enums (AttendingStatusEnum, EventTypeEnum, SchedulableTypeEnum, SponsoringLevelEnum, TalkStatusEnum), and action classes (AttendEventAction, LeaveEventAction). A new migration is added that explicitly drops all events-module database tables: event_submission_speakers, events_agenda, events_sponsors, events_attendees, events_talks, sponsors, and events.

Possibly related PRs

  • he4rt/heartdevs.com#189: Updates the events module's AttendEventAction typing and null-safety, whereas this PR removes the action entirely.

Suggested reviewers

  • gvieira18
  • davicbtoliveira
  • Clintonrocha98
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'remove: legacy events system' clearly and concisely summarizes the main change—removal of the entire legacy events module and its dependencies.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In
`@app-modules/events/database/migrations/2026_05_16_195205_drop_events_module_tables.php`:
- Around line 8-20: The migration defines only up() and drops several tables
(event_submission_speakers, events_agenda, events_sponsors, events_attendees,
events_talks, sponsors, events) so add a down() method to the anonymous
Migration class that reverses those drops; implement down() to recreate each
dropped table (using Schema::create for event_submission_speakers,
events_agenda, events_sponsors, events_attendees, events_talks, sponsors,
events) with the original column definitions (or by copying the table
definitions from the original migrations that created them) so php artisan
migrate:rollback can restore the database state; ensure the method name is
public function down(): void and placed alongside up() in the same class.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Pro

Run ID: 62f13a1e-430f-4633-8f39-1083d2b4d6eb

📥 Commits

Reviewing files that changed from the base of the PR and between ee92602 and 52a0bb9.

📒 Files selected for processing (25)
  • app-modules/events/database/migrations/2025_11_05_191403_create_events_table.php
  • app-modules/events/database/migrations/2025_11_05_192008_create_events_talks_table.php
  • app-modules/events/database/migrations/2025_11_05_192756_create_events_attendees_table.php
  • app-modules/events/database/migrations/2025_11_05_193042_create_sponsors_table.php
  • app-modules/events/database/migrations/2025_11_05_193141_create_events_sponsors_table.php
  • app-modules/events/database/migrations/2025_11_27_132714_add_attend_order_to_events_attendees_table.php
  • app-modules/events/database/migrations/2025_11_27_145728_create_events_agenda_table.php
  • app-modules/events/database/migrations/2025_11_27_153537_remove_starts_at_ends_at_columns_to_events_talks_table.php
  • app-modules/events/database/migrations/2025_11_27_171411_create_event_submission_speakers_table.php
  • app-modules/events/database/migrations/2026_05_16_195205_drop_events_module_tables.php
  • app-modules/events/src/Actions/AttendEventAction.php
  • app-modules/events/src/Actions/LeaveEventAction.php
  • app-modules/events/src/Enums/AttendingStatusEnum.php
  • app-modules/events/src/Enums/EventTypeEnum.php
  • app-modules/events/src/Enums/SchedulableTypeEnum.php
  • app-modules/events/src/Enums/SponsoringLevelEnum.php
  • app-modules/events/src/Enums/Talks/TalkStatusEnum.php
  • app-modules/events/src/Models/EventAgenda.php
  • app-modules/events/src/Models/EventModel.php
  • app-modules/events/src/Models/EventSegment.php
  • app-modules/events/src/Models/EventSubmission.php
  • app-modules/events/src/Models/Pivot/EventAttend.php
  • app-modules/events/src/Models/Pivot/EventSubmissionSpeaker.php
  • app-modules/events/src/Models/Pivot/SponsorAttend.php
  • app-modules/events/src/Models/Sponsor.php
💤 Files with no reviewable changes (24)
  • app-modules/events/database/migrations/2025_11_27_153537_remove_starts_at_ends_at_columns_to_events_talks_table.php
  • app-modules/events/src/Actions/AttendEventAction.php
  • app-modules/events/src/Models/Pivot/EventAttend.php
  • app-modules/events/src/Models/EventSegment.php
  • app-modules/events/src/Models/Pivot/EventSubmissionSpeaker.php
  • app-modules/events/database/migrations/2025_11_27_171411_create_event_submission_speakers_table.php
  • app-modules/events/src/Models/Pivot/SponsorAttend.php
  • app-modules/events/src/Models/EventSubmission.php
  • app-modules/events/src/Enums/SponsoringLevelEnum.php
  • app-modules/events/src/Enums/AttendingStatusEnum.php
  • app-modules/events/database/migrations/2025_11_05_192756_create_events_attendees_table.php
  • app-modules/events/src/Actions/LeaveEventAction.php
  • app-modules/events/database/migrations/2025_11_05_191403_create_events_table.php
  • app-modules/events/database/migrations/2025_11_05_193042_create_sponsors_table.php
  • app-modules/events/src/Models/EventAgenda.php
  • app-modules/events/database/migrations/2025_11_05_192008_create_events_talks_table.php
  • app-modules/events/database/migrations/2025_11_05_193141_create_events_sponsors_table.php
  • app-modules/events/src/Enums/SchedulableTypeEnum.php
  • app-modules/events/src/Enums/Talks/TalkStatusEnum.php
  • app-modules/events/database/migrations/2025_11_27_145728_create_events_agenda_table.php
  • app-modules/events/src/Enums/EventTypeEnum.php
  • app-modules/events/src/Models/EventModel.php
  • app-modules/events/src/Models/Sponsor.php
  • app-modules/events/database/migrations/2025_11_27_132714_add_attend_order_to_events_attendees_table.php

Copy link
Copy Markdown

@GabrielFVDev GabrielFVDev left a comment

Choose a reason for hiding this comment

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

LGTM

@danielhe4rt danielhe4rt merged commit 3c7acdf into 4.x May 16, 2026
6 checks passed
@danielhe4rt danielhe4rt deleted the remove/legacy-events branch May 16, 2026 23:57
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.

3 participants