Skip to content

Conversation

@rchlfryn
Copy link
Collaborator

@rchlfryn rchlfryn commented Nov 12, 2025

Description

Add revalidation and migration for events

Related Issues

Fixes #634
Fixes #635

Key Changes

@github-actions
Copy link

Preview deployment: https://events-revalidation.preview.avy-fx.org

@rchlfryn rchlfryn force-pushed the events-revalidation branch from f99f7a0 to e4c1883 Compare November 17, 2025 23:47
@rchlfryn rchlfryn changed the base branch from events to events-table November 17, 2025 23:47
@rchlfryn rchlfryn changed the title Events revalidation Events revalidation & migration Nov 19, 2025
@busbyk
Copy link
Collaborator

busbyk commented Nov 19, 2025

@rchlfryn I merged in main via events and regenerated the migration since main had the kv migration. I like to just regenerate them in that scenario.

I wrote a diffing script in #689 bc I wanted it to better-interpret the migration diff here. Here are the results:

🔍 Comparing snapshots:
   Old: 20251117_210840_kv.json
   New: 20251119_184931_add_courses_and_events.json

📊 Migration Analysis: 20251119_184931_add_courses_and_events
============================================================

✅ Tables Added (38):
   • _courses_v
   • _courses_v_version_affinity_groups
   • _courses_v_version_mode_of_travel
   • _events_v
   • _events_v_rels
   • _events_v_version_blocks_in_content
   • _events_v_version_mode_of_travel
   • _filterByEventTypes_v
   • _home_pages_v_blocks_event_list
   • _home_pages_v_blocks_event_table
   • _home_pages_v_blocks_single_event
   • _pages_v_blocks_event_list
   • _pages_v_blocks_event_table
   • _pages_v_blocks_single_event
   • _providers_v
   • _providers_v_version_course_types
   • _providers_v_version_states_serviced
   • a3_management
   • courses
   • courses_affinity_groups
   • courses_mode_of_travel
   • event_groups
   • event_tags
   • events
   • events_blocks_in_content
   • events_mode_of_travel
   • events_rels
   • filterByEventTypes
   • home_pages_blocks_event_list
   • home_pages_blocks_event_table
   • home_pages_blocks_single_event
   • pages_blocks_event_list
   • pages_blocks_event_table
   • pages_blocks_single_event
   • providers
   • providers_course_types
   • providers_states_serviced
   • users_rels

📝 Tables Modified (5):
   • _home_pages_v_rels
     + Fields added: event_groups_id, event_tags_id, events_id
   • _pages_v_rels
     + Fields added: event_groups_id, event_tags_id, events_id
   • home_pages_rels
     + Fields added: event_groups_id, event_tags_id, events_id
   • pages_rels
     + Fields added: event_groups_id, event_tags_id, events_id
   • payload_locked_documents_rels
     + Fields added: courses_id, event_groups_id, event_tags_id, events_id, providers_id

@busbyk
Copy link
Collaborator

busbyk commented Nov 19, 2025

Preview failing due to seed script. Fixed: #690

@rchlfryn rchlfryn marked this pull request as ready for review November 20, 2025 03:40
@rchlfryn rchlfryn marked this pull request as draft November 20, 2025 03:42
Base automatically changed from events-table to events November 21, 2025 05:03
@rchlfryn rchlfryn marked this pull request as ready for review November 21, 2025 05:07
@busbyk
Copy link
Collaborator

busbyk commented Nov 21, 2025

Would love to get #697 pulled into here too since it removes some fields on events (cost and capacity).

Copy link
Collaborator

@busbyk busbyk left a comment

Choose a reason for hiding this comment

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

LGTM

@rchlfryn rchlfryn merged commit 0cff54b into events Nov 21, 2025
2 checks passed
@rchlfryn rchlfryn deleted the events-revalidation branch November 21, 2025 18:12
@rchlfryn rchlfryn mentioned this pull request Nov 21, 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.

4 participants