Skip to content

Conversation

@rchlfryn
Copy link
Collaborator

Description

Remove autosave on all collections

Related Issues

Fixes #779

Migration Explanation

Simple migration

@github-actions
Copy link

Migration Safety Check

Found 21 potential issues:

20251219_073149_remove_autosave.ts

Warning (line 5): DROP keyword detected - review for data loss

await db.run(sql`DROP INDEX \`_home_pages_v_autosave_idx\`;`)

Warning (line 6): DROP keyword detected - review for data loss

await db.run(sql`ALTER TABLE \`_home_pages_v\` DROP COLUMN \`autosave\`;`)

Warning (line 6): ALTER keyword detected - review for data loss

await db.run(sql`ALTER TABLE \`_home_pages_v\` DROP COLUMN \`autosave\`;`)

Warning (line 7): DROP keyword detected - review for data loss

await db.run(sql`DROP INDEX \`_pages_v_autosave_idx\`;`)

Warning (line 8): DROP keyword detected - review for data loss

await db.run(sql`ALTER TABLE \`_pages_v\` DROP COLUMN \`autosave\`;`)

Warning (line 8): ALTER keyword detected - review for data loss

await db.run(sql`ALTER TABLE \`_pages_v\` DROP COLUMN \`autosave\`;`)

Warning (line 9): DROP keyword detected - review for data loss

await db.run(sql`DROP INDEX \`_posts_v_autosave_idx\`;`)

Warning (line 10): DROP keyword detected - review for data loss

await db.run(sql`ALTER TABLE \`_posts_v\` DROP COLUMN \`autosave\`;`)

Warning (line 10): ALTER keyword detected - review for data loss

await db.run(sql`ALTER TABLE \`_posts_v\` DROP COLUMN \`autosave\`;`)

Warning (line 11): DROP keyword detected - review for data loss

await db.run(sql`DROP INDEX \`_events_v_autosave_idx\`;`)

Warning (line 12): DROP keyword detected - review for data loss

await db.run(sql`ALTER TABLE \`_events_v\` DROP COLUMN \`autosave\`;`)

Warning (line 12): ALTER keyword detected - review for data loss

await db.run(sql`ALTER TABLE \`_events_v\` DROP COLUMN \`autosave\`;`)

Warning (line 13): DROP keyword detected - review for data loss

await db.run(sql`DROP INDEX \`_providers_v_autosave_idx\`;`)

Warning (line 14): DROP keyword detected - review for data loss

await db.run(sql`ALTER TABLE \`_providers_v\` DROP COLUMN \`autosave\`;`)

Warning (line 14): ALTER keyword detected - review for data loss

await db.run(sql`ALTER TABLE \`_providers_v\` DROP COLUMN \`autosave\`;`)

Warning (line 15): DROP keyword detected - review for data loss

await db.run(sql`DROP INDEX \`_courses_v_autosave_idx\`;`)

Warning (line 16): DROP keyword detected - review for data loss

await db.run(sql`ALTER TABLE \`_courses_v\` DROP COLUMN \`autosave\`;`)

Warning (line 16): ALTER keyword detected - review for data loss

await db.run(sql`ALTER TABLE \`_courses_v\` DROP COLUMN \`autosave\`;`)

Warning (line 17): DROP keyword detected - review for data loss

await db.run(sql`DROP INDEX \`_navigations_v_autosave_idx\`;`)

Warning (line 18): DROP keyword detected - review for data loss

await db.run(sql`ALTER TABLE \`_navigations_v\` DROP COLUMN \`autosave\`;`)

Warning (line 18): ALTER keyword detected - review for data loss

await db.run(sql`ALTER TABLE \`_navigations_v\` DROP COLUMN \`autosave\`;`)

Review these patterns and add backup/restore logic if needed. See docs/migration-safety.md for guidance.

@github-actions
Copy link

Preview deployment: https://remove-autosave.preview.avy-fx.org

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.

We have some query.autosave checks in hooks that we'll want to remove in this PR too.

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.

Remove autosave

3 participants