Skip to content

Remove the old notebook location - #236

Merged
pearmini merged 1 commit into
mainfrom
remove-legacy-notebook-rule
Sep 27, 2026
Merged

pearmini merged 1 commit into
mainfrom
remove-legacy-notebook-rule

Conversation

@pearmini

Copy link
Copy Markdown
Collaborator

All notebooks have been moved from users/{uid}/notebooks to notebooks with a one-off admin script. It moved 10 notebooks from 4 users, and each copy was checked against a backup.

This removes:

  • The Firestore rule for the old location.
  • The migration that ran on every login, which would otherwise fail with a permission error once the rule is gone.

After merging, deploy the rules:

firebase deploy --only firestore:rules

🤖 Generated with Claude Code

All notebooks have been moved from users/{uid}/notebooks to notebooks
with an admin script, so drop the rule for the old location and the
migration that ran on login.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Sep 27, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
recho-notebook Ready Ready Preview Sep 27, 2026 10:16pm UTC

@pearmini
pearmini merged commit 569b7f7 into main Sep 27, 2026
3 checks passed
@pearmini
pearmini deleted the remove-legacy-notebook-rule branch September 27, 2026 22:16

This branch was successfully deployed

1 active deployment
Preview — f4d8754e Deployed Sep 27, 2026 by vercel[bot]
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